[PATCH] D56408: [SimpleLoopUnswitch] Increment stats counter for unswitching switch instruction

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 7 14:38:56 PST 2019


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

One nit, otherwise LGTM.



================
Comment at: llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp:2286
+    ++NumBranches;
+  else if (SI)
+    ++NumSwitches;
----------------
The `if(SI)` is redundant.
Lines 1923 and 2028 above already confirm either `SI` or `BI` exist.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56408





More information about the llvm-commits mailing list