On 5/18/25 2:19 PM, Richard Sandiford wrote:
Jeff Law <jeffreya...@gmail.com> writes:
On 5/16/25 11:32 AM, Jeff Law wrote:
On 5/16/25 11:21 AM, Richard Sandiford wrote:
This pattern used operands[2] to hold the shift amount, even though
the pattern doesn't have an operand 2 (not even as a match_dup).
This caused a build failure with -Werror:
array subscript 2 is above array bounds of ‘rtx_def* [2]’
gcc/
* config/nds32/nds32-intrinsic.md (unspec_get_pending_int): Use
a local variable instead of operands[2].
Obviously OK. IMHO you should just commit this kind of fix.
You might consider looking at pr100837 which looks like it'd be fixed by
this change.
Ah yeah, good spot. I'll add it to the commit message.
Just happened to be grubbing around looking for good bugs for another
intern and stumbled over it...
Jeff
