[PATCH] D61736: [MergeICmps] Simplify the code.

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 15 05:50:32 PDT 2019


gchatelet accepted this revision.
gchatelet added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Transforms/Scalar/MergeICmps.cpp:551
+private:
+  Twine makeTwine(ArrayRef<BCECmpBlock> Comparisons) {
+    assert(!Comparisons.empty() && "no basic block");
----------------
`static`


================
Comment at: llvm/lib/Transforms/Scalar/MergeICmps.cpp:561
+                                     });
+    if (size == 0) {
+      return Twine();
----------------
no braces


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61736





More information about the llvm-commits mailing list