[llvm-commits] [llvm] r164926 - /llvm/trunk/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp

[Michael Liao]

It seems to me a good idea. Attachment is my first draft to add this
support in X86 backend. Thanks for your review.

There should be further changes to clean it up and convert all other
places the encoding bits should be used instead.

Yours
- Michael

On Tue, 2012-10-02 at 09:10 -0700, Jakob Stoklund Olesen wrote:
> On Oct 1, 2012, at 12:33 AM, Craig Topper <craig.topper at gmail.com> wrote:
> 
> > Author: ctopper
> > Date: Mon Oct  1 02:33:27 2012
> > New Revision: 164926
> > 
> > URL: http://llvm.org/viewvc/llvm-project?rev=164926&view=rev
> > Log:
> > Use constants for all return values in switch. Allows clang to optimize it into a lookup table.
> > 
> > Modified:
> >    llvm/trunk/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp
> 
> How about using MCRegisterInfo::getEncodingValue()?
> 
> /jakob
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-register-encoding-support-in-X86-backend.patch
Type: text/x-patch
Size: 37759 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121002/bd57f53a/attachment.bin>