[PATCH] D61517: [InstCombine] Add new combine to add folding.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 3 09:37:02 PDT 2019


lebedev.ri added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/sub.ll:1281
+
+; Check (X | Y) - Y --> (X | Y) ^ Y doesn't happen where (X | Y) has multiple uses
+define i32 @test71(i32 %A, i32 %B) {
----------------
Hm, these are tests from D61307, right?
This comment seems wrong, i think it will trigger there.


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

https://reviews.llvm.org/D61517





More information about the llvm-commits mailing list