[PATCH] D88935: [KnownBits] Add a computeForMul method

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 6 23:31:41 PDT 2020


lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

Looks good to me.



================
Comment at: llvm/unittests/Support/KnownBitsTest.cpp:183
+
+      // ComputedMul should have at most all the information from KnownMul.
+      KnownBits ComputedMul = KnownBits::computeForMul(Known1, Known2);
----------------
// ComputedMul is conservatively correct, but not guaranteed to be precise.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88935



More information about the llvm-commits mailing list