https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95720

--- Comment #13 from Andrea Corallo <andrea.corallo at arm dot com> ---
"aoliva at gcc dot gnu.org" <gcc-bugzi...@gcc.gnu.org> writes:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95720
>
> Alexandre Oliva <aoliva at gcc dot gnu.org> changed:
>
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>   Attachment #48886|0                           |1
>         is obsolete|                            |
>   Attachment #48889|0                           |1
>         is obsolete|                            |
>
> --- Comment #12 from Alexandre Oliva <aoliva at gcc dot gnu.org> ---
> Created attachment 48891
>   --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48891&action=edit
> another try
>
> Sorry about the bugs in the previous patch.
>
> Besides integrating your s/_1/_list/ fix (thanks), I understood why you needed
> the [set $opts], but the intended logic was to have a [set $i] in the previous
> line.  I didn't know about this use for "set", though; I would have gone for
> "eval set opts \$$i" instead.  I liked set opts [set $i] much better.  Thanks
> for cluing me into it!
>
> Would you be so kind as to confirm that this slightly modified version still
> works for you, and that I didn't miss any of your changes?

Hi Alexandre,

I confirm the patch works here.

Just a small nit, whitespace-mode shows me trailing spaces at the end of
gcc_adjust_linker_flags_list.

> Thanks,

Thank you for the fix!

  Andrea

Reply via email to