rsmith added a comment.
In https://reviews.llvm.org/D45174#1055048, @rsmith wrote:
> I wonder if we can delete the `getNonVirtualSize()` check now -- I don't see
> any way that an empty class can have a nonzero nvsize except by this nonempty
> anonymous bit-fields situation.
Yup, looks like:
if (!FoundBase) {
- if (MDCUsesEBO && BaseDecl->isEmpty() &&
- BaseLayout.getNonVirtualSize() == CharUnits::Zero()) {
+ if (MDCUsesEBO && BaseDecl->isEmpty()) {
+ assert(BaseLayout.getNonVirtualSize() == CharUnits::Zero());
BaseOffset = CharUnits::Zero();
} else {
Zero test failures.
Repository:
rC Clang
https://reviews.llvm.org/D45174
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
