[PATCH] D122804: [AMDGPU] Only count global-to-global as indirect accesses
Mikael Holmén via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 5 00:13:51 PDT 2022
uabelho added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUPerfHintAnalysis.cpp:156
auto M = LD->getPointerOperand();
- if (isGlobalAddr(M) || isLocalAddr(M) || isConstantAddr(M)) {
LLVM_DEBUG(dbgs() << " is IA\n");
----------------
foad wrote:
> uabelho wrote:
> > I think this was the last use of the isConstantAddr method. gcc now warns with
> > ```
> > 23:46:06 ../lib/Target/AMDGPU/AMDGPUPerfHintAnalysis.cpp:339:6: warning: 'bool {anonymous}::AMDGPUPerfHint::isConstantAddr(const llvm::Value*) const' defined but not used [-Wunused-function]
> > 23:46:06 339 | bool AMDGPUPerfHint::isConstantAddr(const Value *V) const {
> > 23:46:06 | ^~~~~~~~~~~~~~
> > ```
> > Should the unused isConstantAddr method be removed?
> Sorry about that, but I am hoping to land D83676 soon (it was approved once, just waiting for confirmation that it's still OK) which will start using isConstantAddr again.
Ok, good to know. Thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122804/new/
https://reviews.llvm.org/D122804
More information about the llvm-commits
mailing list