[PATCH] D47194: [AMDGPU] Fixed non-uniform addr64 MUBUF in shader

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/SIInstrInfo.cpp:3667
   if (isMIMG(MI) ||
       (AMDGPU::isShader(MF.getFunction().getCallingConv()) &&
+       (isMUBUF(MI) || isMTBUF(MI))
----------------
I don't like that this was checking the calling convention at all. Can we just eliminate that check altogether?


================
Comment at: lib/Target/AMDGPU/SIInstrInfo.cpp:3669
+       (isMUBUF(MI) || isMTBUF(MI))
+       && !isAddr64(MI))) {
     MachineOperand *SRsrc = getNamedOperand(MI, AMDGPU::OpName::srsrc);
----------------
You should be able to avoid adding isAddr64 by just moving the getNamedOperand call for vaddr below up here


Repository:
  rL LLVM

https://reviews.llvm.org/D47194