- User Since
- Apr 6 2021, 8:14 AM (4 w, 5 d)
Mon, May 3
Plan of action is to follow https://lists.llvm.org/pipermail/cfe-dev/2021-April/068074.html, break a ShadowIntroducingDecl base out of UsingDecl and then add UsingEnumDecl derived from that.
Fri, Apr 30
Thu, Apr 29
Wed, Apr 28
Richard, you are correct, I should have looked more. This update removes that boolean
Tue, Apr 27
Thu, Apr 22
Wed, Apr 21
Updated to fix formatting etc (pretty sure I got clang-format to work this time). I changed the pre-c++20 behaviour to be a warning along the lines you suggested. Trying to move the qualifier checking until after generating the shadow decls died horribly -- the shadow generation presumes the context is sane :(
thanks for the comments. My plan wrt documentation was to address that (and the feature macro) once the patches are in.
Tue, Apr 20
057b6f5d0b3e 2021-04-16 | clang: Update libstdc++ issue workaround
Set 4.8.3 as oldest supported
Mon, Apr 19
Change to 4.8.3 as oldest version
Fri, Apr 16
Ok, hopefully right this time. I went with v4.9, as that's the version Richard mentioned in D100469, I just confused myself with 4.8.3. will email cfe-dev
yeah, I guess that would be a little agressive. See https://reviews.llvm.org/D100680 for a comment update
Added ReleaseNote about libstdc++ -- is that the best place to document such a requirement?
Fix failing test -- I must have confused myself
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?
Wed, Apr 14
Remove orthogonal lbstdc++ FIXME comment