[PATCH] D39064: implement __has_unique_object_representations

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 23 17:36:33 PDT 2017


rnk added inline comments.


================
Comment at: lib/AST/Type.cpp:2226
+        Context.getFieldOffset(*Record->field_begin()));
+    for (const auto *Field : Record->fields()) {
+      if (!Field->getType().hasUniqueObjectRepresentations(Context))
----------------
What about base classes? I think that's where the padding detection is going to get wacky. =/


https://reviews.llvm.org/D39064





More information about the cfe-commits mailing list