[Concurrency] bypass global variable static checking for @TaskLocal property wrapper declarations

Author: sophiapoirier

lib/Sema/TypeCheckConcurrency.cpp

@@ -4445,6 +4445,13 @@ ActorIsolation ActorIsolationRequest::evaluate(
       if (var->isGlobalStorage() && !isActorType) {
         auto *diagVar = var;
         if (auto *originalVar = var->getOriginalWrappedProperty()) {
+          // temporary 5.10 checking bypass for @TaskLocal <rdar://120907014>
+          // TODO: @TaskLocal should be a macro <rdar://120914014>
+          if (auto *classDecl =
+                  var->getInterfaceType()->getClassOrBoundGenericClass();
+              classDecl && classDecl->getName().is("TaskLocal")) {
+            return isolation;
+          }
           diagVar = originalVar;
         }
         if (var->isLet()) {

test/Concurrency/global_variables.swift

@@ -50,6 +50,7 @@ struct TestStatics {
   static var computedProperty: Int { 0 } // computed property that, though static, has no storage so is not a global
   @TestWrapper static var wrapped: Int // expected-error{{static property 'wrapped' is not concurrency-safe because it is non-isolated global shared mutable state}}
   // expected-note@-1{{isolate 'wrapped' to a global actor, or convert it to a 'let' constant and conform it to 'Sendable'}}
+  @TaskLocal static var taskLocal: Int = 0
 }
 
 @TestGlobalActor