[PATCH] D137937: [TableGen] Represent IntrHasSideEffects using inaccessiblemem read+write

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 17 15:38:55 PST 2022


arsenm added inline comments.


================
Comment at: llvm/test/Analysis/GlobalsModRef/nosync_nocallback.ll:34
 check:
   call void @llvm.amdgcn.s.barrier()
   %v = load i32, ptr @G1
----------------
As counter intuitive as it is, I think this is correct. Can you add a second copy of this test. that uses a fence to show it still isn't moved?


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

https://reviews.llvm.org/D137937



More information about the llvm-commits mailing list