[PATCH] D122804: [AMDGPU] Only count global-to-global as indirect accesses

Mon Apr 4 21:42:27 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");
----------------
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?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122804