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

Changpeng Fang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 11 12:34:43 PST 2020


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


================
Comment at: llvm/test/CodeGen/AMDGPU/store-clobbers-load.ll:23
+  ret void
+}
----------------
arsenm wrote:
> Should add some tests with exotic memory writes since this was  bug before
The memory dependence analysis will return isClobber (other than isDef)  dependence for the "exotic memory writes".
While such tests maybe useful, they are beyond this patch.


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

https://reviews.llvm.org/D92951



More information about the llvm-commits mailing list