[PATCH] D62013: [InstSimplify] Add unary fneg to `fsub 0.0, (fneg X) ==> X` transform

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 16 10:52:39 PDT 2019


spatel added inline comments.


================
Comment at: llvm/test/Transforms/InstSimplify/fast-math.ll:224-225
 ;
-  %t1 = fsub float 0.0, %a
-  %ret = fneg nsz float %t1
+  %t1 = fneg float %a
+  %ret = fsub nsz float 0.0, %t1
   ret float %ret
----------------
Better to commit the test fixes ahead of this patch as an NFC step, so we don't lose those diffs in case the code change has to be reverted.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62013





More information about the llvm-commits mailing list