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
Event Timeline
Comment Actions
LGTM
lib/Target/AMDGPU/SIISelLowering.cpp | ||
---|---|---|
271–283 | This should go under all of the cases, since the code assumes MUBUF for all of these address spaces. | |
271–283 | 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 | 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. |
This should go under all of the cases, since the code assumes MUBUF for all of these address spaces.