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.


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

Reply via email to