[PATCH] D56544: [X86][AARCH64] Improve ISD::ABS support

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 10 12:58:19 PST 2019


efriedma added inline comments.


================
Comment at: test/CodeGen/AArch64/iabs.ll:46
+; CHECK-NEXT:  abs d0, d0
+; CHECK-NEXT:  fmov x0, d0
 ; CHECK-NEXT:  ret
----------------
This sequence is legal, but it isn't actually a good idea; the new sequence is one instruction longer, and has much higher latency.

AArch64TargetLowering::LowerINTRINSIC_WO_CHAIN should probably generate a v1i64 ISD::ABS, not an i64 ISD::ABS.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56544





More information about the llvm-commits mailing list