================
@@ -77,6 +77,13 @@ void AMDGPUAnnotateUniformValues::visitLoadInst(LoadInst &I) 
{
   // for memory operations that are live in to entry points only.
   if (!isEntryFunc)
     return;
+
+  // If I is atomic, it might see the effects of concurrent clobbering 
accesses,
+  // so local clobber analysis is not sufficient to ensure that it sees the
+  // initial value.
+  if (I.isAtomic())
----------------
arsenm wrote:

What about volatile? I thought we already handled that correctly, and would 
expect this was an isSimple check. Where is volatile filtered out? 

https://github.com/llvm/llvm-project/pull/219991
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to