The MUBUF addr64 bit has been removed on VI, so we must use FLAT
instructions when the pointer is stored in VGPRs.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM
lib/Target/AMDGPU/SIISelLowering.cpp | ||
---|---|---|
271–283 ↗ | (On Diff #29344) | This should go under all of the cases, since the code assumes MUBUF for all of these address spaces. |
271–283 ↗ | (On Diff #29344) | I'm not sure about using this only for global, but it sort of makes sense |
test/CodeGen/AMDGPU/cgp-addressing-modes.ll | ||
7–11 ↗ | (On Diff #29344) | A test that shows an offset that is sunk on CI and not sunk on VI would be useful. I'm surprised that one of these existing test cases didn't break. |