[PATCH] D121157: [AMDGPU] always use underlying object in the pointsToConstantMemory

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 7 13:45:00 PST 2022


rampitec added a comment.

I am not sure this is a correct approach, but in the getAliasResult() we return tells that global, flat and constant may alias, so it is not ruled out.

This is the regression after D119886 <https://reviews.llvm.org/D119886>. The alternative fix would be to attach amdgpu.noclobber metadata on the load there instead of casting to constant but I cannot rule out such cast may appear for a different reason. I can write a source program like that.


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

https://reviews.llvm.org/D121157



More information about the llvm-commits mailing list