Hi Mark:

Thanks for notifying me, fixed on the trunk :)

[1]
https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=d3651f07bbf56837f019e366b75d01f197dab2f1

On Fri, May 2, 2025 at 4:31 AM Mark Wielaard <m...@klomp.org> wrote:

> Hi Kito,
>
> Unfortunately this breaks bootstrap for riscv:
>
> ../../gcc/gcc/config/riscv/genrvv-type-indexer.cc: In function ‘int
> main(int, const char**)’:
> ../../gcc/gcc/config/riscv/genrvv-type-indexer.cc:302:7: error: this ‘for’
> clause does not guard... [-Werror=misleading-indentation]
>   302 |       for (unsigned eew : EEW_SIZE_LIST)
>       |       ^~~
> ../../gcc/gcc/config/riscv/genrvv-type-indexer.cc:306:9: note: ...this
> statement, but the latter is misleadingly indented as if it were guarded by
> the ‘for’
>   306 |         fprintf (fp, "  /*X2*/ INVALID,\n");
>       |         ^~~~~~~
> cc1plus: all warnings being treated as errors
>
> https://builder.sourceware.org/buildbot/#/builders/338/builds/150
>
> On Wed, Apr 30, 2025 at 05:25:34PM +0800, Kito Cheng wrote:
> [...]
> > > diff --git a/gcc/config/riscv/genrvv-type-indexer.cc
> b/gcc/config/riscv/genrvv-type-indexer.cc
> > > index 6de23cb6e1c..2fd429ad734 100644
> > > --- a/gcc/config/riscv/genrvv-type-indexer.cc
> > > +++ b/gcc/config/riscv/genrvv-type-indexer.cc
> > > @@ -303,6 +303,8 @@ main (int argc, const char **argv)
> > >         fprintf (fp, "  /*UNSIGNED_EEW%d_LMUL1_INTERPRET*/ %s,\n", eew,
> > >                  inttype (eew, LMUL1_LOG2, /* unsigned_p */true).c_str
> ());
> > >
> > > +       fprintf (fp, "  /*X2*/ INVALID,\n");
> > > +
> > >        for (unsigned lmul_log2_offset : {1, 2, 3, 4, 5, 6})
> > >         {
> > >           unsigned multiple_of_lmul = 1 << lmul_log2_offset;
>
> That fprintf line is indented too much.
>
> Cheers,
>
> Mark
>

Reply via email to