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: [PING] [Patch RX] Added warnings of RX variants that does not have hardware FPU support


> I tried to implement your suggestions, but it is issued below build error,
> 
> ../src/binutils-2.23.1/gas/config/rx-parse.y:896.45-46: $4 of 
> `float2_op' has no declared type
> ../src/binutils-2.23.1/gas/config/rx-parse.y:896.61-62: $2 of 
> `float2_op' has no declared type
> 
> Please suggest as I am not familiar with this syntax.

If you rearrange the components of a bison grammar, it renumbers all the $N etc
values.  So, since a new "term" has been introduced, you need to increase
all the numbers by one.

> float2_op
>         : { rx_check_float_support (); }
> 	  '#' EXPR ',' REG
>           { id24 (2, 0x72, sub_op << 4); F ($4, 20, 4); O4 ($2); }

In this example, $1 is the first {...}, #2 is '#', $3 is the EXPR, etc.
So the resulting code would be:

>           { id24 (2, 0x72, sub_op << 4); F ($5, 20, 4); O4 ($3); }

(BTW, that {...} would be $6 ;)


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