[PATCH] D20275: [SimplifyCFG] eliminate switch cases based on known range of switch condition

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Thu May 19 12:19:06 PDT 2016


spatel added inline comments.

================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:3870
@@ +3869,3 @@
+  unsigned ExtraSignBits = ComputeNumSignBits(Cond, DL, 0, AC, SI) - 1;
+  unsigned MinCondWidth = Bits - ExtraSignBits;
+
----------------
davidxl wrote:
> MinCondWidth --> MaxCondWidth?
Yes, that's vague. How about "MaxSignificantBitsInCond"?


http://reviews.llvm.org/D20275





More information about the llvm-commits mailing list