[PATCH] D12249: Remove the final bit test during lowering switch statement if all cases in bit test cover a contiguous range.

Cong Hou via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 21 17:01:03 PDT 2015


congh added a comment.

In http://reviews.llvm.org/D12249#230549, @davidxl wrote:

> I think it should be an optimization that always wins -- the following conditions need to be satisfied:
>
> 1. value range is continuous AND
> 2. min == 0 OR the value range is outside 64 (or whatever largest width supported by the target)


Agree. I will update the patch later.


http://reviews.llvm.org/D12249





More information about the llvm-commits mailing list