diff options
author | Sergei Barannikov <barannikov88@gmail.com> | 2025-04-21 21:42:40 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-04-21 21:42:40 +0300 |
commit | e0c1e23b99e9719d0a01ab7dfc0807d891004bd4 (patch) | |
tree | 0778635fee46048c092f223555eb0120a6a4f7a0 /llvm/lib/Analysis/AssumptionCache.cpp | |
parent | c873ca25976d81f54421d9f4e24c5f700824d0d1 (diff) | |
download | llvm-e0c1e23b99e9719d0a01ab7dfc0807d891004bd4.zip llvm-e0c1e23b99e9719d0a01ab7dfc0807d891004bd4.tar.gz llvm-e0c1e23b99e9719d0a01ab7dfc0807d891004bd4.tar.bz2 |
[TTI] Constify BasicTTIImplBase::thisT() (NFCI) (#136575)
The main change is making `thisT` method `const`, the rest of the
changes is fixing compilation errors (*).
(*) There are two tricky methods, `getVectorInstrCost()` and
`getIntImmCost()`.
They have several overloads; some of these overloads are typically
pulled in to derived classes using the `using` directive, and then
hidden by methods in the derived class.
The compiler does not complain if the hiding methods are not marked as
`const`, which means that clients will use the methods from the base
class. If after this change your target fails cost model tests, this
must be the reason. To resolve the issue you need to make all hiding
overloads `const`. See the second commit in this PR.
Pull Request: https://github.com/llvm/llvm-project/pull/136575
Diffstat (limited to 'llvm/lib/Analysis/AssumptionCache.cpp')
0 files changed, 0 insertions, 0 deletions