[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