The return should be a break

snarang181 · snarang181 · commit 7d6b60899419 · 2025-06-25T09:21:15.000-04:00
diff --git a/clang/lib/Sema/SemaTypeTraits.cpp b/clang/lib/Sema/SemaTypeTraits.cpp
@@ -2376,7 +2376,7 @@ static void DiagnoseNonStandardLayoutReason(Sema &SemaRef, SourceLocation Loc,
           << FirstField;
 
       // No need to check further fields, as we already found mixed access.
-      return;
+      break;
     }
   }
   if (hasMultipleDataBaseClassesWithFields(D)) {

Original file line number	Diff line number	Diff line change
`@@ -2376,7 +2376,7 @@ static void DiagnoseNonStandardLayoutReason(Sema &SemaRef, SourceLocation Loc,`
`2376`	`2376`	`<< FirstField;`
`2377`	`2377`
`2378`	`2378`	`// No need to check further fields, as we already found mixed access.`
`2379`		`- return;`
	`2379`	`+ break;`
`2380`	`2380`	`}`
`2381`	`2381`	`}`
`2382`	`2382`	`if (hasMultipleDataBaseClassesWithFields(D)) {`