[PATCH] D154189: [clang][Interp] Implement zero-init of record types

[Timm Bäder via Phabricator via cfe-commits]

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