diff options
author | Stanislav Mekhanoshin <rampitec@users.noreply.github.com> | 2023-12-18 10:58:50 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-12-18 10:58:50 -0800 |
commit | 94230ce548c2f69db4cf9ef92ccb51d59030e7e3 (patch) | |
tree | 3f061d0f5f8c637d99950b01b55bf401cd1781ce /libcxx/include/__algorithm/search.h | |
parent | 4f54d71501815877898aaa6d764b8468d0708ed6 (diff) | |
download | llvm-94230ce548c2f69db4cf9ef92ccb51d59030e7e3.zip llvm-94230ce548c2f69db4cf9ef92ccb51d59030e7e3.tar.gz llvm-94230ce548c2f69db4cf9ef92ccb51d59030e7e3.tar.bz2 |
[AMDGPU] Fix lack of LDS DMA check in the AA handling (#75249)
SIInstrInfo::areMemAccessesTriviallyDisjoint does a DS offset checks,
but does not account for LDS DMA instructions. Added these checks.
Without it code falls through and returns true which is wrong. As a
result mayAlias would always return false for LDS DMA and a regular LDS
instruction or 2 LDS DMA instructions.
At the moment this is NFCI because we do not use this AA in a context
which may touch LDS DMA instructions. This is also unreacheable now
because of the ordered memory ref checks just above in the function and
LDS DMA is marked as volatile. This volatile marking is removed in PR
#75247, therefore I'd submit this check before #75247.
Diffstat (limited to 'libcxx/include/__algorithm/search.h')
0 files changed, 0 insertions, 0 deletions