probinson added a comment.

In https://reviews.llvm.org/D46218#1081933, @rjmccall wrote:

> I wonder if that was originally just an oversight that got turned into 
> official policy.  Anyway, if it's the policy, it's what we have to do; LGTM.


I think it's actually correct behavior.  Why would an attribute on a derived 
class be allowed to alter the layout of a base class?  It would mean you can't 
convert Derived* to Base* safely.


Repository:
  rL LLVM

https://reviews.llvm.org/D46218



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D46218: P... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... John McCall via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... Paul Robinson via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits

Reply via email to