[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