[clang] [Clang][Sema] Detect inline namespace ambiguity on namespace redefinition (PR #234689)

[Chen Miao via cfe-commits]

================
@@ -12077,8 +12077,20 @@ Decl *Sema::ActOnStartNamespaceDef(Scope *NamespcScope,
       PrevNS = getStdNamespace();
       IsStd = true;
       AddToKnown = !IsInline;
+    } else if (R.isAmbiguous()) {
+      // Handle an ambiguous namespace reopening (often after inlining).
----------------
ChenMiaoi wrote:

Are these comments necessary? Isn’t it already clear once we reach 
`isAmbiguous`?

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