This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [PATCH] [ARC] Add new ARC EM opcodes.


Patch committed.

Thank you Nick for reviewing it,
Claudiu

> -----Original Message-----
> From: Nick Clifton [mailto:nickc@redhat.com]
> Sent: Tuesday, March 15, 2016 11:30 AM
> To: Claudiu Zissulescu; binutils@sourceware.org
> Cc: Francois.Bedard@synopsys.com
> Subject: Re: [PATCH] [ARC] Add new ARC EM opcodes.
> 
> Hi Claudiu,
> 
> > I've update the patch to use a number of macros to
> > generate those special instructions.
> 
> Thanks - although it looks like it took more work to do this,
> than was in the original patch.  Sorry about that. I was hoping
> that the macros would save time and space not need more of them.
> 
> > I also added a couple of tests which are checking for the
> > macro expansion sanity.
> 
> Excellent.
> 
> > Moreover, I plan to use those macros in a  new feature
> > which I still need to add and test.
> 
> Good idea.
> 
> One small point...  why are macros like EXTINSN2OP and EXTINSN3OP
> (and their supporting macros) defined in arc-opc.c ?  Do you intend
> to provide other definitions of these macros in other source files ?
> The reason I ask, it that it seems to me that if the macros are
> only going to be defined once, then the appropriate place would be
> in a header, either arc-ext-tbl.h or arc.h.
> 
> If you do want to leave the definitions in arc-opc.c, (and I am OK
> with it if you do), then please could you add a small comment to
> arc-ext-tbl.h noting where the EXTINSN<n>OP macros are defined.
> 
> Approved with this change, or with a change to move the definitions
> into a header.
> 
> Cheers
>   Nick
> 
> 
> 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]