On Mon, Feb 03, 2025 at 10:36:15AM +0100, Richard Biener wrote:
> > --- a/gcc/config/mips/mips.cc
> > +++ b/gcc/config/mips/mips.cc
> > @@ -9631,7 +9631,13 @@ mips_expand_block_move (rtx dest, rtx src, rtx
> > length, rtx alignment)
> > {
> > if (ISA_HAS_COPY)
> > return mips16_expand_copy (dest, src, length, alignment);
> > - else if (INTVAL (length) <= MIPS_MAX_MOVE_BYTES_PER_LOOP_ITER)
> > + else if (INTVAL (length) <= MIPS_MAX_MOVE_BYTES_PER_LOOP_ITER
> > + /* We increase slightly the maximum number of bytes in
> > + a straight-line block if the source and destination
> > + are aligned to the register width. */
> > + || (!optimize_size
> > + && INTVAL (alignment) == UNITS_PER_WORD
> > + && INTVAL (length) <= MIPS_MAX_MOVE_MEM_STRAIGHT))
The formatting here doesn't follow the coding conventions.
Dunno if this is in the MUA say replacing a tab with spaces, but unlikely,
e.g. the || line should be indented by one tab and 7 spaces to go under
INTVAL in else if line.
See https://gcc.gnu.org/contribute.html#standards for details.
Jakub
