Message ID | B5E67142681B53468FAF6B7C3135656244155B69@hhmail02.hh.imgtec.org |
---|---|
State | New |
Headers | show |
> > We could add -mflip-micromips complementing -mflip-mips16 and use > > that for testing too. Chances are it'd reveal further issues. > > Looking at how > > -mflip-mips16 has been implemented it does not appear to me adding > > -mflip-micromips would be a lot of effort. > > I'm in favour of adding such a switch since the testsuite doesn't cover > a mixture of MIPS and microMIPS code. It certainly seems that we need a bit more coverage here in order that we can mostly stick to testing one or two MIPS configurations per commit. We'll have some MIPS machines in the compile farm shortly which may allow us to at least do the full all-config build of the toolchain more easily even if that doesn't extend to testing all the configs. > > Regards, > Robert > > gcc/ > * config/mips/mips.h (micromips_globals): Declare. OK, thanks. Matthew
> > gcc/ > > * config/mips/mips.h (micromips_globals): Declare. > > OK, thanks. > > Matthew Committed as r223438. Robert
diff --git a/gcc/config/mips/mips.h b/gcc/config/mips/mips.h index 0ea4e6d..85c8a97 100644 --- a/gcc/config/mips/mips.h +++ b/gcc/config/mips/mips.h @@ -3108,6 +3108,7 @@ extern const struct mips_cpu_info *mips_arch_info; extern const struct mips_cpu_info *mips_tune_info; extern unsigned int mips_base_compression_flags; extern GTY(()) struct target_globals *mips16_globals; +extern GTY(()) struct target_globals *micromips_globals; #endif /* Enable querying of DFA units. */