[PATCH] D43825: [MergeICmp] Fix a bug in MergeICmp that can lead to a block being processed more than once.

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 28 01:09:23 PST 2018


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

Thanks for the fix !



================
Comment at: lib/Transforms/Scalar/MergeICmps.cpp:581
+      //
+      // This is an uncommon case, so we bail.
+      return false;
----------------
Can you add a debug line:

DEBUG(dbgs() << "skip: non-constant value not from cmp\n");


Repository:
  rL LLVM

https://reviews.llvm.org/D43825





More information about the llvm-commits mailing list