[PATCH] D118661: [AMDGPU] Check atomics aliasing in the clobbering annotation

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUAnnotateUniformValues.cpp:51
     AU.addRequired<MemorySSAWrapperPass>();
     AU.setPreservesAll();
  }
----------------
Should add an addUsedIfAvailable call for AA here. But I think MemorySSA already requires it, so you could make it a hard requirement here too?


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

https://reviews.llvm.org/D118661