On Tue, Apr 26, 2016 at 9:04 AM, Bernd Schmidt <bschm...@redhat.com> wrote:
> On 04/25/2016 07:55 PM, Jason Merrill wrote:
>>
>> On 04/25/2016 05:07 AM, Bernd Schmidt wrote:
>>>
>>> +        if (TREE_CODE (arg2) == CONST_DECL)
>>> +          arg2 = DECL_INITIAL (arg2);
>>> +        int literal_mask = ((!!integer_zerop (arg1) << 1)
>>> +                    | (!!integer_zerop (arg2) << 2));
>>
>>
>> Are you deliberately treating an enumerator as a literal 0?  I'd think
>> we should set literal_mask before stripping CONST_DECL.  OK with that
>> change.
>
>
> Just to be sure, is that OK for everything or just the C++ parts?

Everything.

Jason

Reply via email to