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

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 27 10:41:11 PST 2019


evandro added inline comments.


================
Comment at: lib/Target/AArch64/AArch64ISelLowering.cpp:5407
+  // vs. adrp+ldr (it's one instruction longer, but the movw+movk is fused).
+  // So we limit up to 2 instrdduction at most.
+  if (!IsLegal && (VT == MVT::f64 || VT == MVT::f32)) {
----------------
Perhaps you could check for AArch64Subtarget::hasFuseLiterals() and emit up to 5 instructions, thus including f64, unless optimizing for size.


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

https://reviews.llvm.org/D58460





More information about the llvm-commits mailing list