inclyc added a comment.
Hi @aaron.ballman seems we are failing on recent change
2fc5a3410087c209567c7e4a2c457b6896c127a3
<https://reviews.llvm.org/rG2fc5a3410087c209567c7e4a2c457b6896c127a3>
/* The DR asked a question about whether defining a new type within offsetof
* is allowed. C2x N2350 made this explicitly undefined behavior, but Clang
* has always supported defining a type in this location, and GCC also
* supports it.
*/
(void)__builtin_offsetof(struct S { int a; }, a);
Do you think this patch should still be merged into mainline? Or do we turn
this error into a warning?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133574/new/
https://reviews.llvm.org/D133574
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
