================
@@ -10566,6 +10637,10 @@ Sema::ActOnFunctionDeclarator(Scope *S, Declarator &D, 
DeclContext *DC,
       }
     }
 
+    if (getLangOpts().BoundsSafety) {
----------------
rapidsna wrote:

I don't think guarding on bounds safety extension makes much sense here. The 
whole point of this warning is to collect data to help us make decisions about 
name lookup in bounds safety-related parameters — so if we gate it behind the 
extension, we won't actually get that data.

Also, the bounds safety extension flag isn't even functional, so the guard 
would be pretty meaningless anyway.

I'd just remove it.

https://github.com/llvm/llvm-project/pull/181550
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to