[PATCH] D87342: WIP: Plumb TargetTransformInfo into computeKnownBits

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 8 21:05:56 PDT 2020


qcolombet added inline comments.


================
Comment at: llvm/unittests/Analysis/ValueTrackingTest.cpp:1202
+                           F->front().getTerminator(), /*DT=*/nullptr,
+                           /*ORE=*/nullptr, /*UseInstrInfo=*/true, &TTI);
+  // We know of one less zero because 512 may have produced a 1 that
----------------
qcolombet wrote:
> Note: This test illustrates how it would work with a custom TTI but is not commit-able as is because it relies on `BasicTTIImplBase`, which is part of the CodeGen library.
Fixed that part.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87342



More information about the llvm-commits mailing list