[PATCH] D47203: [LowerSwitch] Fixed faulty PHI node in switch default block

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 9 03:06:41 PDT 2018


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

LGTM with nit



================
Comment at: lib/Transforms/Utils/LowerSwitch.cpp:517
+      // one entry in the PHI nodes.
+      for (unsigned i = 0 ; i < (MaxPop - 1) ; ++i)
+        PopSucc->removePredecessor(OrigBlock);
----------------
i should be capitalized


Repository:
  rL LLVM

https://reviews.llvm.org/D47203





More information about the llvm-commits mailing list