[cxx-interop] Avoid creating duplicate unsafe attributes

Gabor Horvath · Gabor Horvath · commit 303aa1b79189 · 2025-01-24T12:46:07.000Z
diff --git a/lib/ClangImporter/ImportDecl.cpp b/lib/ClangImporter/ImportDecl.cpp
@@ -4124,6 +4124,7 @@ namespace {
           // annotated nor is it marked noescape.
           auto attr = new (ASTContext) UnsafeAttr(/*implicit=*/true);
           result->getAttrs().add(attr);
+          break;
         }
       }
       Impl.diagnoseTargetDirectly(decl);

Original file line number	Diff line number	Diff line change
`@@ -4124,6 +4124,7 @@ namespace {`
`4124`	`4124`	`// annotated nor is it marked noescape.`
`4125`	`4125`	`auto attr = new (ASTContext) UnsafeAttr(/implicit=/true);`
`4126`	`4126`	`result->getAttrs().add(attr);`
	`4127`	`+ break;`
`4127`	`4128`	`}`
`4128`	`4129`	`}`
`4129`	`4130`	`Impl.diagnoseTargetDirectly(decl);`