[PATCH] D87342: Allow targets to augment computeKnownBits with their analysis using TargetTransformInfo

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 15 16:31:23 PDT 2020


qcolombet updated this revision to Diff 292053.
qcolombet added a comment.

- Fix a few call sites where I was passing `TTI` instead of the boolean for `UseInstrInfo` (it doesn't help that the compiler didn't warm on these)

Now, it should really be NFC, unless you set the `TTI`, like in the unit test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87342

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/include/llvm/Analysis/ValueTracking.h
  llvm/include/llvm/CodeGen/BasicTTIImpl.h
  llvm/include/llvm/Support/KnownBits.h
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/lib/Analysis/ValueTracking.cpp
  llvm/lib/Support/KnownBits.cpp
  llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp
  llvm/unittests/Analysis/ValueTrackingTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D87342.292053.patch
Type: text/x-patch
Size: 61191 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200915/a53befbb/attachment-0001.bin>


More information about the llvm-commits mailing list