OpenE2K/gcc
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

PR target/98878 - Incorrect multilib list for riscv*-rtems

Browse Source 
- Multi-lib only check the default argument is appeared in the multilib
   flag list, but we didn't check the case that the flag is required but
   default argument didn't provide.

 - For example riscv*-rtems has a multilib set:

     rv32imafd/ilp32d;@march=rv32imafd@mabi=ilp32d

   And when we set the default argument of arch to rv32imafdc and ABI to
   ilp32d, gcc will check the ilp32d is matched, but it didn't check
   rv32imafd is not provided by default arguments, so it should not
   reuse default library.

 - The side effcet of this patch is csky-elf target will increase the
   number of multilib from 54 to 59, but it's because genmultilib didn't
   know the default argument for multilib, so I think it could be
   improved in future.

gcc/ChangeLog:

	* gcc.c (print_multilib_info): Check all required argument is provided
	by default arg.
This commit is contained in:
Kito Cheng 2021-01-28 18:29:53 +08:00
parent 8d0737d8f4
commit 072f20c555
1 changed files with 21 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
gcc/gcc.c
View File

@ -9909,12 +9909,14 @@ print_multilib_info (void)
last_path_len = p - this_path;
}
/* If this directory requires any default arguments, we can skip
it. We will already have printed a directory identical to
this one which does not require that default argument. */
/* If all required arguments are default arguments, and no default
arguments appear in the ! argument list, then we can skip it.
We will already have printed a directory identical to this one
which does not require that default argument. */
if (! skip)
{
const char *q;
bool default_arg_ok = false;
q = p + 1;
while (*q != ';')
@ -9946,16 +9948,29 @@ print_multilib_info (void)
list. */
if (not_arg)
{
skip = 0;
default_arg_ok = false;
break;
}
else
skip = 1;
default_arg_ok = true;
}
else if (!not_arg)
{
/* Stop checking if any required argument is not provided by
default arguments. */
default_arg_ok = false;
break;
}
if (*q == ' ')
++q;
}
/* Make sure all default argument is OK for this multi-lib set. */
if (default_arg_ok)
skip = 1;
else
skip = 0;
}
if (! skip)