[PATCH] D91701: [AMDGPU] Implement flat scratch init for pal

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 18 09:38:52 PST 2020


rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIFrameLowering.cpp:320
+  MBB.addLiveIn(GitPtrLo);
+  BuildMI(MBB, I, DL, SMovB32, TargetLo).addReg(GitPtrLo);
+}
----------------
.addReg() on the next line.


================
Comment at: llvm/lib/Target/AMDGPU/SIFrameLowering.cpp:320
+  MBB.addLiveIn(GitPtrLo);
+  BuildMI(MBB, I, DL, SMovB32, TargetLo).addReg(GitPtrLo);
+}
----------------
rampitec wrote:
> .addReg() on the next line.
Isn't this a first subreg def, so this one shall have superreg def?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91701



More information about the llvm-commits mailing list