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. Richard
