[PATCH C++] Fix PR77489 -- mangling of discriminator >= 10

Jason Merrill jason@redhat.com
Wed Jan 18 15:05:00 GMT 2017


On Wed, Jan 18, 2017 at 9:23 AM, Markus Trippelsdorf
<markus@trippelsdorf.de> wrote:
> On 2017.01.18 at 09:11 -0500, Jason Merrill wrote:
>> On Wed, Jan 18, 2017 at 3:55 AM, Markus Trippelsdorf
>> <markus@trippelsdorf.de> wrote:
>> > On 2017.01.17 at 13:26 -0500, Jason Merrill wrote:
>> >> On Thu, Jan 12, 2017 at 2:36 AM, Markus Trippelsdorf
>> >> <markus@trippelsdorf.de> wrote:
>> > +      if (abi_version_at_least (11) && discriminator - 1 >= 10)
>> > +       {
>> > +         write_char ('_');
>> > +         if (abi_warn_or_compat_version_crosses (11))
>> > +           G.need_abi_warning = 1;
>>
>> This check should be outside the abi_version_at_least block; we want
>> to warn if -fabi-version=10 and -Wabi=11.
>
> +      if (abi_warn_or_compat_version_crosses (11))
> +       G.need_abi_warning = 1;
>        write_char ('_');
>        if (abi_version_at_least (11) && discriminator - 1 >= 10)

Ah, but it does need to be controlled by the second part of this test;
we only want the warning if the discriminator will be two digits.

Jason



More information about the Gcc-patches mailing list