[C++ Patch] Fix grokbitfield location

Jason Merrill jason@redhat.com
Wed Dec 5 22:52:00 GMT 2018


On 12/5/18 5:34 PM, Paolo Carlini wrote:
> Hi,
> 
> On 05/12/18 20:31, Jason Merrill wrote:
>> On 12/5/18 7:45 AM, Paolo Carlini wrote:
>>> Hi,
>>>
>>> as mentioned in one of my last patches, we can now improve this 
>>> location. Note: in the same function there are a few further issues 
>>> which I mean to incrementally fix (eg, the diagnostics for 
>>> warn_if_not_aligned ICEs for unnamed bit-fields). Tested x86_64-linux.
>> As long as we're messing with this diagnostic, let's also print the 
>> type in question.
> 
> Agreed. Thus I tested on x86_64-linux the below.
> 
> Thanks, Paolo.
> 
> /////////////////////
> 
> 
> -  if (!INTEGRAL_OR_ENUMERATION_TYPE_P (TREE_TYPE (value))
> -      && (INDIRECT_TYPE_P (value)
> -          || !dependent_type_p (TREE_TYPE (value))))
> +  if (!INTEGRAL_OR_ENUMERATION_TYPE_P (type)
> +      && (INDIRECT_TYPE_P (value) || !dependent_type_p (type)))

Hmm, surely this should be INDIRECT_TYPE_P (type).  OK with that change.

Jason



More information about the Gcc-patches mailing list