[PATCH] D99861: [Clang] Record tokens in attribute arguments for user-defined C++/C2x attributes

[Nathan Sidwell via Phabricator via cfe-commits]

urnathan added a comment.

See also https://bugs.llvm.org/show_bug.cgi?id=46446.  when I first fell into this issue, I did think it was trying to save the token stream as this patch is doing.  Neat I thought :)  although I'm a clang weenie, saving the tokens is putting this into deferred-parse territory, which makes me nervous.  Wouldn't it be better for the ParsedAttrInfo objects to determine whether and how to parse their arguments.  They could do so immediately, or save tokens, or whatever on a per-attribute per-argument basis.  Isn't that more flexible?  Add some ParsedAttrInfo defaults for the default cxx11, gnu & clang flavours of attributes?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99861/new/

https://reviews.llvm.org/D99861