This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch ARM/ configury] Add fall-back check for gnu_unique_object
> > Ping - ?http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00549.html
>
> And now really add Paolo and DJ.
+ [.type foo, '$target_type_format_char'gnu_unique_object],,
This un-quoting looks incorrect if you don't know what's going on
under the hood, but I don't see a clean way around it. A suitable
comment would be appropriate.
+target_type_format_char="@"
+ target_type_format_char="%"
Since the string always has "special" characters, it's likely that
single quotes are more appropriate here. The two characters in the
patch don't care, but some future porter might naively do "$" and
wonder why (or worse, not wonder why) it doesn't work right.
Other than that it looks OK to me, assuming you tested it on all the
relevent targets (i.e. arm and not-arm).