[llvm] [AMDGPU] Folding imm offset in more cases for scratch access (PR #70634)

via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 06:29:18 PST 2023


================
@@ -1860,7 +1905,7 @@ bool AMDGPUDAGToDAGISel::SelectScratchSVAddr(SDNode *N, SDValue Addr,
           CurDAG->getTargetConstant(RemainderOffset, SDLoc(), MVT::i32));
         VAddr = SDValue(VMov, 0);
----------------
ruiling wrote:

Are you asking "Will we possibly put negative value in VAddr"? I think the answer is no. As there is a check `isUInt<32>(RemainderOffset)` a few lines above.

https://github.com/llvm/llvm-project/pull/70634


More information about the llvm-commits mailing list