[Concurrency] Rename AbstractClosureExpr::getActorIsolation to

getClosureActorIsolation.

This is preparation for changing AbstractClosureExpr to store
ActorIsolation instead of ClosureActorIsolation, and convert to
ClosureActorIsolation when needed to allow incrementally updating
callers. This change is NFC.

(cherry picked from commit 4b23564)

Author: hborla

include/swift/AST/Expr.h

@@ -3948,7 +3948,9 @@ class AbstractClosureExpr : public DeclContext, public Expr {
   /// returns nullptr if the closure doesn't have a body
   BraceStmt *getBody() const;
 
-  ClosureActorIsolation getActorIsolation() const { return actorIsolation; }
+  ClosureActorIsolation getClosureActorIsolation() const {
+    return actorIsolation;
+  }
 
   void setActorIsolation(ClosureActorIsolation actorIsolation) {
     this->actorIsolation = actorIsolation;

lib/AST/ASTDumper.cpp

@@ -2592,7 +2592,7 @@ class PrintExpr : public ExprVisitor<PrintExpr> {
     PrintWithColorRAII(OS, DiscriminatorColor)
       << " discriminator=" << E->getRawDiscriminator();
 
-    switch (auto isolation = E->getActorIsolation()) {
+    switch (auto isolation = E->getClosureActorIsolation()) {
     case ClosureActorIsolation::Nonisolated:
       break;
 

lib/AST/Decl.cpp

@@ -10209,7 +10209,7 @@ bool VarDecl::isSelfParamCaptureIsolated() const {
     }
 
     if (auto closure = dyn_cast<AbstractClosureExpr>(dc)) {
-      switch (auto isolation = closure->getActorIsolation()) {
+      switch (auto isolation = closure->getClosureActorIsolation()) {
       case ClosureActorIsolation::Nonisolated:
       case ClosureActorIsolation::GlobalActor:
         return false;

lib/AST/Expr.cpp

@@ -2014,7 +2014,7 @@ Expr *AbstractClosureExpr::getSingleExpressionBody() const {
 
 ClosureActorIsolation
 swift::__AbstractClosureExpr_getActorIsolation(AbstractClosureExpr *CE) {
-  return CE->getActorIsolation();
+  return CE->getClosureActorIsolation();
 }
 
 llvm::function_ref<ClosureActorIsolation(AbstractClosureExpr *)>

lib/IDE/CompletionLookup.cpp

@@ -777,7 +777,7 @@ void CompletionLookup::analyzeActorIsolation(
       if (isolation != ClosureActorIsolations.end()) {
         return isolation->second;
       } else {
-        return CE->getActorIsolation();
+        return CE->getClosureActorIsolation();
       }
     };
     auto contextIsolation = getActorIsolationOfContext(

lib/SILGen/SILGenProlog.cpp

@@ -1380,7 +1380,7 @@ void SILGenFunction::emitProlog(
     }
   } else if (auto *closureExpr = dyn_cast<AbstractClosureExpr>(FunctionDC)) {
     bool wantExecutor = F.isAsync() || wantDataRaceChecks;
-    auto actorIsolation = closureExpr->getActorIsolation();
+    auto actorIsolation = closureExpr->getClosureActorIsolation();
     switch (actorIsolation.getKind()) {
     case ClosureActorIsolation::Nonisolated:
       break;
@@ -1593,7 +1593,7 @@ void SILGenFunction::emitHopToActorValue(SILLocation loc, ManagedValue actor) {
   }
   auto isolation =
       getActorIsolationOfContext(FunctionDC, [](AbstractClosureExpr *CE) {
-        return CE->getActorIsolation();
+        return CE->getClosureActorIsolation();
       });
   if (isolation != ActorIsolation::Nonisolated
       && isolation != ActorIsolation::Unspecified) {

lib/Sema/CSSimplify.cpp

@@ -2968,7 +2968,7 @@ static bool okToRemoveGlobalActor(ConstraintSystem &cs,
     // So, I expect the existing isolation to always be set to the default.
     // If the assertion below starts tripping, then this ad-hoc inference
     // is no longer needed!
-    auto existingIso = ace->getActorIsolation();
+    auto existingIso = ace->getClosureActorIsolation();
     if (existingIso != ClosureActorIsolation()) {
       assert(false && "somebody set the closure's isolation already?");
       return existingIso;