[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 added inline comments. Comment at: clang/test/SemaCXX/offsetof.cpp:106 +int x3[__builtin_offsetof(struct X2, X2::static_a) == 0 ? 1 : -1]; // expected-error{{no member named 'static_a'}} +int x4[__builtin_offsetof(struct X2, X2::X2) == 0 ? 1 : -1]; //

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 added inline comments. Comment at: clang/test/SemaCXX/offsetof.cpp:106 +int x3[__builtin_offsetof(struct X2, X2::static_a) == 0 ? 1 : -1]; // expected-error{{no member named 'static_a'}} +int x4[__builtin_offsetof(struct X2, X2::X2) == 0 ? 1 : -1]; //

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 added a comment. Is there any code handling the nested qualifier that I can reference? I tried to modify the `OffsetOfNode`, which allows me to handle the basic template case but still failed on the nested case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 added inline comments. Comment at: clang/test/SemaCXX/offsetof.cpp:106 +int x3[__builtin_offsetof(struct X2, X2::static_a) == 0 ? 1 : -1]; // expected-error{{no member named 'static_a'}} +int x4[__builtin_offsetof(struct X2, X2::X2) == 0 ? 1 : -1]; //

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 marked an inline comment as done. yichi170 added inline comments. Comment at: clang/docs/ReleaseNotes.rst:59 +- Improved ``__builtin_offsetof`` support, allowing qualified name in member designator. This fixes [Issue

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 updated this revision to Diff 548632. yichi170 marked 4 inline comments as done. yichi170 added a comment. Applied the suggestions. Thanks for giving me feedback! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157201/new/

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 updated this revision to Diff 547764. yichi170 marked an inline comment as not done. yichi170 added a comment. updated test in `test/SemaCXX/offsetof.cpp` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157201/new/

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 added inline comments. Comment at: clang/test/Sema/offsetof.c:79 +int x2[__builtin_offsetof(struct X2, X2::a) == 0 ? 1 : -1]; +int x3[__builtin_offsetof(struct X2, X2::X2) == 0 ? 1 : -1]; // expected-error{{no member named 'X2'}} + Fznamznon wrote: >

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 updated this revision to Diff 547703. yichi170 added a comment. updated ReleaseNotes and applied suggestions. The related issue #63443 is still not fixed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 added inline comments. Comment at: clang/lib/Sema/SemaExpr.cpp:16696-16697 + + IdentifierInfo *II = RD->getIdentifier(); + if (II == OC.U.IdentInfo && OC.isQualifier) +continue; tbaeder wrote: > I'll fix it along with the

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 updated this revision to Diff 547542. yichi170 added a comment. [Clang] Support qualified name as member designator in offsetof Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157201/new/ https://reviews.llvm.org/D157201 Files:

[PATCH] D157201: [Clang] Support qualified name as member designator in offsetof

yichi170 created this revision. Herald added a project: All. yichi170 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D157201 Files: clang/include/clang/Sema/Sema.h