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

Michael Liao michael.liao at intel.com
Tue Oct 2 11:41:08 PDT 2012


On Tue, 2012-10-02 at 11:32 -0700, Jakob Stoklund Olesen wrote:
> On Oct 2, 2012, at 11:25 AM, Michael Liao <michael.liao at intel.com> wrote:
> 
> > It seems to me a good idea. Attachment is my first draft to add this
> > support in X86 backend. Thanks for your review.
> 
> Very nice.
> 
> I think it would be cleaner to use an X86Reg class that forces the HWEncoding to be set for all registers. See ARMReg in ARMRegisterInfo.td.\\

Yeah, that will make the code even cleaner. I will refine that.

Yours
- Michael

> 
> /jakob
> 





More information about the llvm-commits mailing list