[llvm-commits] [llvm] r43918 - in /llvm/trunk/lib/Target/X86: X86RegisterInfo.cpp X86RegisterInfo.td

[Dale Johannesen]

On Nov 9, 2007, at 9:03 AM, Chris Lattner wrote:
> On Nov 9, 2007, at 8:55 AM, Evan Cheng wrote:
>> Frankly I don't think the issue is worth spending this much time on.
>> Whatever works and maintainable is good.
>
> I'd prefer to use one switch statement form or the other.  The
> compiler should be able to turn trivial switches like that into an
> array lookup.  Note that it doesn't currently, but adding this
> shouldn't be hard.
>
> -Chris

OK, I don't see a consensus in our future, so I've checked in what I  
think
is the best thing.  If anyone dislikes this enough to rewrite it, I  
will not object.