[PATCH] Added instruction combine to transform few more negative values addition to subtraction

Benjamin Kramer benny.kra at gmail.com
Wed May 14 08:49:40 PDT 2014


================
Comment at: lib/Transforms/InstCombine/InstCombineAddSub.cpp:929
@@ +928,3 @@
+      C1->getValue() == ~C2->getValue()) {
+    return Builder->CreateAnd(Z, C1);
+  }
----------------
Can you return the fully transformed value? I find it confusing to have this split in half. You can create both instructions with IRBuilder and use ReplaceInstUsesWith at the call site.

http://reviews.llvm.org/D3733






More information about the llvm-commits mailing list