[PATCH] D37019: Add select simplifications

Michael Berg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 22 15:27:15 PDT 2017


mcberg2017 added a comment.

Should i add a fsub case in the tests?



================
Comment at: lib/Transforms/InstCombine/InstCombineMulDivRem.cpp:749
+            Value *V0 = Builder.CreateFMul(A1, A2);
+            Value *V1 = Builder.CreateFMul(B1, B2);
+            return SelectInst::Create(C1, V0, V1);
----------------
Ditto here regarding fastmath


https://reviews.llvm.org/D37019





More information about the llvm-commits mailing list