[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