[PATCH] D92951: AMDGPU: If a store defines (alias) a load, it clobbers the load.

[Changpeng Fang via Phabricator via llvm-commits]

cfang marked an inline comment as done.
cfang added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUAnnotateUniformValues.cpp:115
+        // Store defines the load and thus clobbers it.
+        (Q.isDef() && isa<StoreInst>(Q.getInst())))
       return true;
----------------
arsenm wrote:
> Specifically checking for store instructions is going to be wrong since many other instruction types can also write to memory
Changed to mayWriteToMemory.


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

https://reviews.llvm.org/D92951