[PATCH] D58460: [AArch64] Optimize floating point materialization

Adhemerval Zanella via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 26 10:52:39 PST 2019


zatrazz updated this revision to Diff 188415.
zatrazz marked 3 inline comments as done.
zatrazz added a comment.

New revision based on previous comments. I refactored the logic used on isFPImmLegal to evaluate whether to materialize the FP constant or not by adding a new function on common aarch64 code, AArch64_AM::getExpandImmCost. To avoid code replication I refactored the code by moving some definitions from AArch64ExpandPseudoInsts.cpp.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58460/new/

https://reviews.llvm.org/D58460

Files:
  lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp
  lib/Target/AArch64/AArch64ISelLowering.cpp
  lib/Target/AArch64/MCTargetDesc/AArch64AddressingModes.h
  test/CodeGen/AArch64/arm64-fp-imm.ll
  test/CodeGen/AArch64/fpimm.ll
  test/CodeGen/AArch64/literal_pools_float.ll
  test/CodeGen/AArch64/win_cst_pool.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58460.188415.patch
Type: text/x-patch
Size: 26186 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190226/d25cc352/attachment.bin>


More information about the llvm-commits mailing list