tbaeder added inline comments.
================
Comment at: clang/test/AST/Interp/records.cpp:939
+ };
+#endif
+
----------------
aaron.ballman wrote:
> I think we should have more test coverage. I mentioned a few above, but other
> things to test would be inner classes (and anonymous members), like:
> ```
> struct S {
> struct {
> int x, y;
> } v;
> union {
> float a;
> int b;
> };
> };
> ```
> and member pointers (because those aren't simple pointers):
> ```
> struct A {
> void func();
> };
> struct S {
> void (A::*ptr)();
> };
> ```
member pointers are also not handled yet.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154189/new/
https://reviews.llvm.org/D154189
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
