[MiscDiagnostics] Don't diagnose recursive property access in distributed thunks

Distributed thunks have to refer to the "local" version
of the property directly with implicit `self.` base.

Author: xedin

lib/Sema/MiscDiagnostics.cpp

@@ -1424,7 +1424,10 @@ static void diagRecursivePropertyAccess(const Expr *E, const DeclContext *DC) {
           if (MRE->getAccessSemantics() == AccessSemantics::Ordinary) {
             bool shouldDiagnose = false;
             // Warn about any property access in the getter.
-            if (Accessor->isGetter())
+            //
+            // Distributed thunks have to refer to the "local" version
+            // of the property directly with implicit `self.` base.
+            if (Accessor->isGetter() && !Accessor->isDistributedThunk())
               shouldDiagnose = !isStore;
             // Warn about stores in the setter, but allow loads.
             if (Accessor->isSetter())