Re: [PATCH] PR target/98878 - Incorrect multilib list for riscv*-rtems

2021-02-04 Thread Kito Cheng via Gcc-patches
Thanks Jim, committed with your revision of comment :) On Fri, Feb 5, 2021 at 11:28 AM Jim Wilson wrote: > > On Thu, Feb 4, 2021 at 2:02 AM Kito Cheng wrote: >> >> * gcc.c (print_multilib_info): Check all required argument is >> provided >> by default arg. > > > This looks OK to

Re: [PATCH] PR target/98878 - Incorrect multilib list for riscv*-rtems

2021-02-04 Thread Jim Wilson
On Thu, Feb 4, 2021 at 2:02 AM Kito Cheng wrote: > * gcc.c (print_multilib_info): Check all required argument is > provided > by default arg. > This looks OK to me, but... > > - /* If this directory requires any default arguments, we can skip > + /* If this directory r

Re: [PATCH] PR target/98878 - Incorrect multilib list for riscv*-rtems

2021-02-04 Thread Sebastian Huber
On 04/02/2021 11:01, Kito Cheng wrote: gcc/ChangeLog: * gcc.c (print_multilib_info): Check all required argument is provided by default arg. Thanks, with this patch the riscv-rtems* target works again. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim

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

2021-02-04 Thread Kito Cheng
- 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