[llvm-commits] [Patch] Replace switches with lookup tables (PR884)

[Sean Silva]

Great!

I hope to see the small-integer followup patch soon!

btw, do you have any pre/post code/data size measurements? I would
gather them myself, but my machine takes 20+ minutes per build :(

--Sean Silva

On Thu, Sep 6, 2012 at 6:15 AM, Hans Wennborg <hans at chromium.org> wrote:
> Thanks everyone! Landed r163302. (And fixed in r163305.)
>
>  - Hans
>
> On Thu, Sep 6, 2012 at 1:44 AM, Jim Grosbach <grosbach at apple.com> wrote:
>> No additional review comments from me. Just want to add a big +1 that I'm looking forward to this. I, too, have been frustrated that we don't make this transform.
>>
>> -Jim
>>
>> On Sep 4, 2012, at 4:08 AM, Benjamin Kramer <benny.kra at gmail.com> wrote:
>>
>>>
>>> On 04.09.2012, at 11:25, Hans Wennborg <hans at chromium.org> wrote:
>>>
>>>> Thanks for the review! New patch attached.