Liu
2011-May-07 09:47 UTC
[LLVMdev] [MIPS] some type errors in the commentary of *.td file.
On Sat, May 7, 2011 at 12:33 PM, Eric Christopher <echristo at apple.com> wrote:> > On May 5, 2011, at 11:26 PM, Liu wrote: > >> some some type errors in the commentary of *.td file. >> such as: >> lib/Target/Mips/MipsInstrFormats.td >> MipsRegisterInfo.td - Mips Register defs >> it should be >> MipsInstrFormats.td - Mips Instruction Formats >> >> lib/Target/Mips/MipsInstrInfo.td >> MipsInstrInfo.td - Mips Register defs >> it should be >> MipsInstrInfo.td - Target Description for Mips Target >> >> and some thing I think it is wrong in lib/Target/Mips/MipsInstrFPU.td. >> >> Will the patch be OK? >> >> Best regards. >> >> --Liu >> <Mips-td-comment-type-error.patch> > > Looks fine. Do you need someone to apply this for you? > > -eric >Hi Eric I'm so glad someone apply this to me. Will you? Thanks. --Liu
Eric Christopher
2011-May-09 18:23 UTC
[LLVMdev] [MIPS] some type errors in the commentary of *.td file.
On May 7, 2011, at 2:47 AM, Liu wrote:> On Sat, May 7, 2011 at 12:33 PM, Eric Christopher <echristo at apple.com> wrote: >> >> On May 5, 2011, at 11:26 PM, Liu wrote: >> >>> some some type errors in the commentary of *.td file. >>> such as: >>> lib/Target/Mips/MipsInstrFormats.td >>> MipsRegisterInfo.td - Mips Register defs >>> it should be >>> MipsInstrFormats.td - Mips Instruction Formats >>> >>> lib/Target/Mips/MipsInstrInfo.td >>> MipsInstrInfo.td - Mips Register defs >>> it should be >>> MipsInstrInfo.td - Target Description for Mips Target >>> >>> and some thing I think it is wrong in lib/Target/Mips/MipsInstrFPU.td. >>> >>> Will the patch be OK? >>> >>> Best regards. >>> >>> --Liu >>> <Mips-td-comment-type-error.patch> >> >> Looks fine. Do you need someone to apply this for you? >> >> -eric >> > > Hi Eric > > I'm so glad someone apply this to me. Will you? > Thanks.Applied here: [yendi:Data/sources/llvm] echristo% svn ci lib/Target/Mips/ Sending lib/Target/Mips/MipsInstrFPU.td Sending lib/Target/Mips/MipsInstrFormats.td Sending lib/Target/Mips/MipsInstrInfo.td Transmitting file data ... Committed revision 131086. -eric
Liu
2011-May-09 23:30 UTC
[LLVMdev] [MIPS] some type errors in the commentary of *.td file.
On Tuesday, May 10, 2011, Eric Christopher <echristo at apple.com> wrote:> > On May 7, 2011, at 2:47 AM, Liu wrote: > >> On Sat, May 7, 2011 at 12:33 PM, Eric Christopher <echristo at apple.com> wrote: >>> >>> On May 5, 2011, at 11:26 PM, Liu wrote: >>> >>>> some some type errors in the commentary of *.td file. >>>> such as: >>>> lib/Target/Mips/MipsInstrFormats.td >>>> MipsRegisterInfo.td - Mips Register defs >>>> it should be >>>> MipsInstrFormats.td - Mips Instruction Formats >>>> >>>> lib/Target/Mips/MipsInstrInfo.td >>>> MipsInstrInfo.td - Mips Register defs >>>> it should be >>>> MipsInstrInfo.td - Target Description for Mips Target >>>> >>>> and some thing I think it is wrong in lib/Target/Mips/MipsInstrFPU.td. >>>> >>>> Will the patch be OK? >>>> >>>> Best regards. >>>> >>>> --Liu >>>> <Mips-td-comment-type-error.patch> >>> >>> Looks fine. Do you need someone to apply this for you? >>> >>> -eric >>> >> >> Hi Eric >> >> I'm so glad someone apply this to me. Will you? >> Thanks. > > Applied here: > > [yendi:Data/sources/llvm] echristo% svn ci lib/Target/Mips/ > Sending lib/Target/Mips/MipsInstrFPU.td > Sending lib/Target/Mips/MipsInstrFormats.td > Sending lib/Target/Mips/MipsInstrInfo.td > Transmitting file data ... > Committed revision 131086. > > -eric >Hi Eric, Thank you for review and apply, nice man. Best wishes, --Liu
Liu
2011-May-10 01:51 UTC
[LLVMdev] [MIPS] some type errors in the commentary of *.td file.
On Tuesday, May 10, 2011, Eric Christopher <echristo at apple.com> wrote:> > On May 7, 2011, at 2:47 AM, Liu wrote: > >> On Sat, May 7, 2011 at 12:33 PM, Eric Christopher <echristo at apple.com> wrote: >>> >>> On May 5, 2011, at 11:26 PM, Liu wrote: >>> >>>> some some type errors in the commentary of *.td file. >>>> such as: >>>> lib/Target/Mips/MipsInstrFormats.td >>>> MipsRegisterInfo.td - Mips Register defs >>>> it should be >>>> MipsInstrFormats.td - Mips Instruction Formats >>>> >>>> lib/Target/Mips/MipsInstrInfo.td >>>> MipsInstrInfo.td - Mips Register defs >>>> it should be >>>> MipsInstrInfo.td - Target Description for Mips Target >>>> >>>> and some thing I think it is wrong in lib/Target/Mips/MipsInstrFPU.td. >>>> >>>> Will the patch be OK? >>>> >>>> Best regards. >>>> >>>> --Liu >>>> <Mips-td-comment-type-error.patch> >>> >>> Looks fine. Do you need someone to apply this for you? >>> >>> -eric >>> >> >> Hi Eric >> >> I'm so glad someone apply this to me. Will you? >> Thanks. > > Applied here: > > [yendi:Data/sources/llvm] echristo% svn ci lib/Target/Mips/ > Sending lib/Target/Mips/MipsInstrFPU.td > Sending lib/Target/Mips/MipsInstrFormats.td > Sending lib/Target/Mips/MipsInstrInfo.td > Transmitting file data ... > Committed revision 131086. > > -eric >Hi Eric, Thank you for review and apply, nice man. Best wishes, --Liu
Seemingly Similar Threads
- [LLVMdev] [MIPS] some type errors in the commentary of *.td file.
- [LLVMdev] [MIPS] some type errors in the commentary of *.td file.
- [LLVMdev] [MIPS] some type errors in the commentary of *.td file.
- [LLVMdev] HELP - tblgen -gen-asm-matcher restrictions on .td content
- [LLVMdev] HELP - tblgen -gen-asm-matcher restrictions on .td content