[clang] [Clang][Sema] Fix malformed AST for anonymous class access in template. (PR #90842)
Erich Keane via cfe-commits
cfe-commits at lists.llvm.org
Thu May 2 14:25:26 PDT 2024
erichkeane wrote:
>This is the option I've decided to go with here. There's a slight wrinkle in that we create a LookupResult that claims we looked up the unnamed field for the anonymous class -- even though we would obviously never be able to look up an unnamed field. I think this is defensible and still better than the other alternatives, but I would welcome feedback on this from others who know the code better.
I actually prefer option#2. This doesn't really do what it should, so I think we should just make sure we materialize the temporary if necessary in `RebuildMemberExpr`.
https://github.com/llvm/llvm-project/pull/90842
More information about the cfe-commits
mailing list