AMDGPUPerfHintAnalysis: don't treat constant loads as large stride

[foad]

Summary

In the "large stride" heuristic ignore loads from the constant address
space (as well as local address space). K$ behavior is very different
from L0$ so it doesn't make much sense to use the same heuristic for
them.

[arsenm]

Testcase?

[rampitec]

LGTM, but needs a test.

[foad]

Add a test case.

[rampitec]

This is LGTM, but needs test fix first of course.

[arsenm]

Typo Perh in commit message

[arsenm]

llvm/lib/Target/AMDGPU/AMDGPUPerfHintAnalysis.cpp

345–346

This isn't really a good implementation. We use scalar loads in more cases and constant address space isn't a guarantee of SMEM loads

[foad]

This is LGTM, but needs test fix first of course.

I think the test is OK now I have rebased on D122804?

[arsenm]

LGTM I guess, but isConstantAddr really should be fixed. CONSTANT_ADDRESS isn't sufficient or even that helpful for knowing if this will use scalar loads

llvm/lib/Target/AMDGPU/AMDGPUPerfHintAnalysis.cpp

 AMDGPUPerfHint::makeMemAccessInfo(Instruction *Inst) const {
   MemAccessInfo MAI;
   const Value *MO = getMemoryInstrPtr(Inst);
 
   LLVM_DEBUG(dbgs() << "[isLargeStride] MO: " << *MO << '\n');
   // Do not treat local-addr memory access as large stride.
   if (isLocalAddr(MO))
     return MAI;
+  // Do not treat constant-addr memory access as large stride because K$
+  // behavior is very different from L0$.
+  if (isConstantAddr(MO))
+    return MAI;
 
   MAI.V = MO;
   MAI.Base = GetPointerBaseWithConstantOffset(MO, MAI.Offset, *DL);
   return MAI;
 }
 
 bool AMDGPUPerfHint::isConstantAddr(const Value *V) const {
   if (auto PT = dyn_cast<PointerType>(V->getType())) {
     unsigned As = PT->getAddressSpace();
     return As == AMDGPUAS::CONSTANT_ADDRESS ||
            As == AMDGPUAS::CONSTANT_ADDRESS_32BIT;
   }
   return false;
 }
 
 bool AMDGPUPerfHint::MemAccessInfo::isLargeStride(
     MemAccessInfo &Reference) const {
 
   if (!Base || !Reference.Base || Base != Reference.Base)