[PATCH] Fix source range of defaulted/deleted member declarations

[Eli Bendersky]

In http://reviews.llvm.org/D8465#144183, @jpienaar wrote:

> In http://reviews.llvm.org/D8465#144141, @eliben wrote:
>
> > In http://reviews.llvm.org/D8465#144139, @jpienaar wrote:
> >
> > > Looks good. Is it better to adjust the range here or n ParseCXXInlineMethodDef?
> >
> >
> > The problem with ParseCXXInlineMethodDef is that we no longer have access to these tokens in it. ParseCXXInlineMethodDef expects to see a { for a def or ; for a =default/=delete method. So we'd have to push DeleteOrDefaultEndLoc through to ParseCXXInlineMethodDef - it would result in an additional argument, more code and I'm not sure what gain.
>
>
> OK, I thought KWLoc that is set in ParseCXXInlineMethodDef could be used to get it.


Yes, indeed. Updated the patch.

Thanks


http://reviews.llvm.org/D8465

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/