[PATCH] D109054: Small improvement in SimplifyCFG.cpp

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 1 22:03:44 PDT 2021


mkazantsev requested changes to this revision.
mkazantsev added a comment.
This revision now requires changes to proceed.

I'm OK with the code change, but please add some informative description on what this patch is about.



================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:6602
+          else {
+            Value* Cond = BI->getCondition();
+            if (BI->getSuccessor(0) == BB)
----------------
You can also add a comment on why is that made.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109054



More information about the llvm-commits mailing list