[PATCH] D150636: [LLVM][Uniformity] Propagate divergence only along divergent outputs.

Tue May 16 02:33:41 PDT 2023

sameerds added inline comments.

================
Comment at: llvm/test/Analysis/UniformityAnalysis/AMDGPU/MIR/always-uniform-gmir.mir:100

-# FIXME :: BELOW INLINE ASM SHOULD BE DIVERGENT
 ---
----------------
foad wrote:
> Just checking my understanding: this should have said "FIXME: first output of below inline asm should be uniform"?
That is my understanding too! After recent changes, it is certainly true that the s32 output should be uniform, and this current change ensures that its use in the COPY instruction also remains uniform. If I revert the current change but keep this test, then the CHECK-NOT fails for %4.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150636