On 12/2/20 6:06 PM, Jim Wilson wrote:
> On Tue, Dec 1, 2020 at 3:24 PM Jeff Law via Gcc-patches
> <gcc-patches@gcc.gnu.org <mailto:gcc-patches@gcc.gnu.org>> wrote:
>
>
>     Kito's recent change to multilib handling seems to have exposed a
>     latent
>     mcore bug.
>
>     The mcore 210 does not support little endian.  Yet we try to build a
>     mcore-210 little-endian multilibs.
>
>     I don't know why this wasn't failing before, but clearly it's not
>     supposed to work.  This patch adjusts the multilib set to not generate
>     that particular library configuration.  The net result is mcore's
>     libgcc
>     builds again, as does newlib.
>
>
> You have two default args in MULTILIB_DEFAULTS and that was never
> supported by the print_multilib_info function.  It only handled one
> default arg correctly.  It must have been working by accident before
> Kito fixed it.  But since we know what the underlying problem is, we
> can check for targets with more than one option in MULTILIB_DEFAULTS. 
> Looks like targets that might be affected are csky, m32r, mcore, mips,
> nds32, riscv, rs6000/sysv*, and sh.  We know that riscv works
> correctly as we checked that.  And you just fixed mcore.  We should
> probably check the others.  They may or may not be OK with Kito's patch.
m32r, mcore, nds32 are at least building.  THe others my tester builds
as linux configurations, I think with --disable-multilib, so I don't
have any data on them.

jeff

Reply via email to