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
