[clang] [llvm] [Clang] Fix definition of layout-compatible to ignore empty classes (PR #92103)

[Vlad Serebrennikov via cfe-commits]

================
@@ -561,6 +561,42 @@ void CXXRecordDecl::addedClassSubobject(CXXRecordDecl 
*Subobj) {
     data().StructuralIfLiteral = false;
 }
 
+const CXXRecordDecl *CXXRecordDecl::getStandardLayoutBaseWithFields() const {
+#ifndef NDEBUG
+  {
+    assert(
+        isStandardLayout() &&
+        "getStandardLayoutBaseWithFields called on a non-standard-layout 
type");
----------------
Endilll wrote:

I think this assert should be on the default code path, not just for expensive 
checks.

https://github.com/llvm/llvm-project/pull/92103
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits