OK.
On Wed, Nov 14, 2018 at 4:26 AM Paolo Carlini <paolo.carl...@oracle.com> wrote:
>
> Hi,
>
> On 14/11/18 01:30, Jason Merrill wrote:
> > On 11/12/18 6:39 AM, Paolo Carlini wrote:
> >> 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.
> > Great, I was going to ask about that. Can I see that patch, then?
>
> Thanks, well I didn't post it because it is a trivial incremental change
> vs the posted one. Attached.
>
> Paolo.
>
> //////////////////
>
