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/ _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
