[PATCH] D69358: AMDGPU: Fix the broken dominator tree

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 24 15:14:16 PDT 2019


arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/SIInstrInfo.cpp:4459
     for (auto &Succ : RemainderBB->successors()) {
-      if (MDT->dominates(&MBB, Succ)) {
+      if (Succ != &MBB && MDT->dominates(&MBB, Succ)) {
         MDT->changeImmediateDominator(Succ, RemainderBB);
----------------
->properlyDominates?


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

https://reviews.llvm.org/D69358





More information about the llvm-commits mailing list