[Bug testsuite/95720] [11 Regression] New dump output filename strategy invalidates tests
andrea.corallo at arm dot com
gcc-bugzilla@gcc.gnu.org
Mon Jul 20 07:44:58 GMT 2020
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-bugzilla@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
More information about the Gcc-bugs
mailing list