[PATCH] [ARC] Add new ARC EM opcodes.
Claudiu Zissulescu
Claudiu.Zissulescu@synopsys.com
Thu Mar 10 11:30:00 GMT 2016
Hi Nick,
Thank you for the clarifications.
> >> +++ b/include/opcode/arc.h
> >> @@ -66,7 +66,8 @@ typedef enum
> >> SHFT1,
> >> SHFT2,
> >> SWAP,
> >> - SP
> >> + SP,
> >> + QUARKSE
> >> } insn_subclass_t;
>
> This enum was alpha- sorted before this change. Is there any particular
> reason for
> removing that property ?
No, I will add it to its proper position.
>
>
> >> +/* QuarkSE specific instructions. */
> >> +{"dsp_fp_div", 0x382A0000, 0xF8FF0000, ARC_OPCODE_ARCv2EM,
> FLOAT,
> >> QUARKSE, { RA, RB, RC }, { C_F }},
>
> Given that there is so much repetition in these new entries, wouldn't it make
> sense
> to use a couple of macros to automate most of the fields ? There is less
> chance for
> typographical errors that way too.
I will try to get some sort of macro expansion for those regular instructions. Usually, I am getting those instructions in a tab-form from hardware team. So, it is pretty easy for me just to dump them as they arrive.
Best,
Claudiu
More information about the Binutils
mailing list