[PATCH] D109288: [AggresiveInstCombine] Add wrapper calls for `KnownBits` computing

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 5 07:00:04 PDT 2021


lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/AggressiveInstCombine/AggressiveInstCombineInternal.h:111
+    return llvm::computeKnownBits(V, DL, /*Depth=*/0, /*AC=*/nullptr,
+                                  /*CtxI=*/cast<Instruction>(CurrentTruncInst),
+                                  &DT);
----------------
Ugh. And phab ate my comment yet again.
Let's not add CtxI here, but in a patch that requires it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109288



More information about the llvm-commits mailing list