[PATCH] D118458: [AMDGPU] Add test for a problem with noclobber metadata

Mon Jan 31 13:25:10 PST 2022

rampitec added a comment.

In D118458#3283506 <https://reviews.llvm.org/D118458#3283506>, @foad wrote:

> In D118458#3280607 <https://reviews.llvm.org/D118458#3280607>, @rampitec wrote:
>
>> Ugh... These are different arguments which may alias. I guess you could do the same with a single gep being reused for loads and stores...
>
> Right, but if a store is followed by a load from the exact same address, I thought some other pass might optimise the load away by replacing it with the value that was stored.

That's a slippery ground. It should be better to find a way to attach it right to a load.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118458