[PATCH] Insert poisoned paddings between fields in C++ classes.

Alexey Samsonov vonosmas at gmail.com
Wed Oct 15 18:08:59 PDT 2014


================
Comment at: lib/AST/Decl.cpp:3642
@@ +3641,3 @@
+    ReasonToReject = "standard layout";
+  else if (II && Blacklist.isBlacklistedType(II->getName(), "field-padding"))
+    ReasonToReject = "blacklisted by type";
----------------
This looks weird: II->getName() is neither mangled nor qualified (what would it return for the record declared inside the namespaces)? Consider testing smth. like printQualifiedName().

http://reviews.llvm.org/D5687






More information about the cfe-commits mailing list