[PATCH] D139780: [AMDGPU] Unify divergent nodes if the PostDom tree has one root

Tue Jan 3 21:07:47 PST 2023

arsenm added inline comments.

================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUUnifyDivergentExitNodes.cpp:109
   // This is a cluster of orthogonal Transforms
   AU.addPreservedID(LowerSwitchID);
   FunctionPass::getAnalysisUsage(AU);
----------------
ruiling wrote:
> gandhi21299 wrote:
> > arsenm wrote:
> > > We should have a required LowerSwitchID too
> > I will have a seperate patch for that, it seems to be causing difficulties when the pass manager schedules UnifyDivergentExitNodes.
> I think for function pass dependency or pass ordering, I still prefer they are managed by compiler developer. If I remember correctly, the new pass manager does not support dependency between function passes?
The important part is verification. We shouldn't have arbitrary pass contracts not enforced by a verifier

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139780