[PATCH] D32946: [InstCombine] Merge duplicate functionality between InstCombine and ValueTracking

[Takuto Ikuta via Phabricator via llvm-commits]

takuto.ikuta added inline comments.


================
Comment at: lib/Analysis/ValueTracking.cpp:3523
+    MaxRHS.clearSignBit();
+    APInt Result = std::move(MaxLHS) + std::move(MaxRHS);
+    return Result.isSignBitClear();
----------------
craig.topper wrote:
> takuto.ikuta wrote:
> > Can I ask you to remove `std::move`s?
> > These have no meaning here.
> They do have meaning. APInt's operator+ is setup to reuse one of the APInts storage for this case.
Ah, I got. sorry for wrong comment.



https://reviews.llvm.org/D32946