[C++ Patch] Fix two grokdeclarator locations

Paolo Carlini paolo.carlini@oracle.com
Mon Nov 12 11:39:00 GMT 2018


Hi again,

On 08/11/18 10:26, Paolo Carlini wrote:
> Hi,
>
> two additional grokdeclarator locations that we can easily fix by 
> using declarator->id_loc. Slightly more interesting, testing revealed 
> a latent issue in the make_id_declarator uses: 
> cp_parser_member_declaration wasn't setting declarator->id_loc, thus I 
> decided to add a location_t parameter to make_id_declarator itself and 
> adjust all the callers. Tested x86_64-linux.

PS: In my local tree I have the cp_parser_objc_class_ivars change using 
token->location instead of UNKNOWN_LOCATION, thus all the 
make_id_declarator calls should be completely fine location-wise.

Paolo.



More information about the Gcc-patches mailing list