[BoundsSafety] Delay processing of bounds attrs in templates

This is a cherry-pick of change that was landed to `stable/20240723`
(swiftlang#9929)
but not landed on the `next` branch.

Aside from resolving merge conflicts it was also necessary to modify
the `clang/test/BoundsSafety/(AST|Sema)/value-dependence.cpp` tests
because they fail to work when the new bounds checks are enabled
(rdar://150044760). To workaround that the test has been made to disable
the new `return_size` bounds check when `-fbounds-safety` is enabled
with C++.

---
* [BoundsSafety] Delay processing of bounds attrs in templates

Dynamic bounds attributes do not handle value dependent arguments. To
enable wider interop with C++ code bases we delay the processing of
these attributes inside templated contexts. Instead we apply the new
type while instantiating the function.

One issue with this approach is that instantiation happens after parsing
is finished, and the Scope information we use to prevent attributes from
referring to arguments from outer scopes is only available during
parsing. These diagnostics were emitted at the end of the type
processing. In templated contexts we instead perform that analysis
during parsing, and delay the rest of the processing. To avoid churn in
unrelated tests the rest of the attributes keep their processing as is,
until we've investigated what impact hoisting it would have on the user
experience.

 Conflicts:
	clang/include/clang/Sema/Sema.h
	clang/lib/Sema/SemaDecl.cpp
	clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
	clang/test/SemaCXX/warn-unsafe-buffer-usage-count-attributed-pointer-argument.cpp

rdar://150694971

(cherry picked from commit c87bc75)

Author: hnrklssn

clang/include/clang/Sema/Sema.h

@@ -2792,7 +2792,8 @@ class Sema final : public SemaBase {
                                     AttributeCommonInfo::Kind Kind,
                                     Expr *AttrArg, SourceLocation Loc,
                                     SourceRange Range, StringRef DiagName,
-                                    bool OriginatesInAPINotes = false);
+                                    bool OriginatesInAPINotes = false,
+                                    bool InInstantiatedTemplate = false);
   /* TO_UPSTREAM(BoundsSafety) OFF*/
 
 

clang/lib/Sema/BoundsSafetySuggestions.cpp

@@ -1594,6 +1594,10 @@ void UnsafeOperationVisitor::
       Current = PE->getSubExpr();
       continue;
     }
+    if (const auto *BCE = dyn_cast<BoundsCheckExpr>(Current)) {
+      Current = BCE->getGuardedExpr();
+      continue;
+    }
     if (const auto *CE = dyn_cast<CastExpr>(Current)) {
       if (CE->getCastKind() == clang::CK_BoundsSafetyPointerCast) {
         // Found a cast we might want to warn about.

clang/lib/Sema/DynamicCountPointerAssignmentAnalysis.cpp

@@ -3485,6 +3485,9 @@ namespace clang {
 
 void DynamicCountPointerAssignmentAnalysis::run() {
   AnalysisDeclContext AC(/* AnalysisDeclContextManager */ nullptr, dcl);
+  if (dcl->isTemplated()) // delay processing until template has been
+                          // instantiated
+    return;
 
   CFG *cfg = AC.getCFG();
   if (!cfg)

clang/lib/Sema/SemaDecl.cpp

@@ -17763,6 +17763,11 @@ Decl *Sema::ActOnFinishFunctionBody(Decl *dcl, Stmt *Body,
   if (FD)
     CheckImmediateEscalatingFunctionDefinition(FD, getCurFunction());
 
+  /*TO_UPSTREAM(BoundsSafety) ON*/
+  if (LangOpts.BoundsSafety)
+    DynamicCountPointerAssignmentAnalysis(*this, dcl).run();
+  /*TO_UPSTREAM(BoundsSafety) OFF*/
+
   if (!IsInstantiation)
     PopDeclContext();
 
@@ -17785,11 +17790,6 @@ Decl *Sema::ActOnFinishFunctionBody(Decl *dcl, Stmt *Body,
   if (FD && !FD->isDeleted())
     checkTypeSupport(FD->getType(), FD->getLocation(), FD);
 
-  /*TO_UPSTREAM(BoundsSafety) ON*/
-  if (LangOpts.BoundsSafety)
-    DynamicCountPointerAssignmentAnalysis(*this, dcl).run();
-  /*TO_UPSTREAM(BoundsSafety) OFF*/
-
   return dcl;
 }