[TSAR, Analysis, Server] Update utils.

If P is a module pass, an alias tree cannot be requested.
So, it should be specified explicitly.

Author: kaniandr

include/tsar/Analysis/Memory/DIClientServerInfo.h

@@ -89,9 +89,8 @@ struct DIClientServerInfo {
 /// it returns information from client.
 struct DIMemoryClientServerInfo : public DIClientServerInfo {
   /// Obtain analysis from client and server.
-  ///
-  /// \post if alias tree is not available on client this class becomes invalid.
-  DIMemoryClientServerInfo(llvm::Pass &P, llvm::Function &F);
+  DIMemoryClientServerInfo(DIAliasTree &ClientDIAT, llvm::Pass &P,
+    llvm::Function &F);
 
   /// Return client-to-server memory pair for a specified estimate memory on
   /// client.
@@ -116,11 +115,6 @@ struct DIMemoryClientServerInfo : public DIClientServerInfo {
   /// client, this function returns analysis results from client.
   DIDependenceSet *findFromClient(const llvm::Loop &L);
 
-  /// Return true if data is available.
-  bool isValid() const noexcept {
-    return DIAT && DIDepInfo && ClientDIAT;
-  }
-
   /// Return true if data is available.
   operator bool () const noexcept { return isValid(); }
 

lib/Analysis/Clang/CanonicalLoop.cpp

@@ -694,7 +694,8 @@ bool CanonicalLoopPass::runOnFunction(Function &F) {
   auto &SE = getAnalysis<ScalarEvolutionWrapperPass>().getSE();
   auto &DT = getAnalysis<DominatorTreeWrapperPass>().getDomTree();
   DeclarationMatcher LoopMatcher = makeLoopMatcher();
-  DIMemoryClientServerInfo DIMInfo(*this, F);
+  DIMemoryClientServerInfo DIMInfo(
+      getAnalysis<DIEstimateMemoryPass>().getAliasTree(), *this, F);
   CanonicalLoopLabeler Labeler(RgnInfo, LoopInfo, MemInfo, ATree,
       TLI, SE, DT, DIMInfo, &mCanonicalLoopInfo);
   auto &Context = FuncDecl->getASTContext();

lib/Analysis/Memory/DIClientServerInfo.cpp

@@ -89,16 +89,9 @@ DIClientServerInfo::DIClientServerInfo(llvm::Pass &P, llvm::Function &F) {
   }
 }
 
-DIMemoryClientServerInfo::DIMemoryClientServerInfo(
-    llvm::Pass &P, llvm::Function &F) : DIClientServerInfo(P, F) {
-  if (auto *DIATP = P.getAnalysisIfAvailable<DIEstimateMemoryPass>())
-    if (DIATP->isConstructed())
-      ClientDIAT = &DIATP->getAliasTree();
-    else
-      return;
-  else
-    return;
-}
+DIMemoryClientServerInfo::DIMemoryClientServerInfo(DIAliasTree &ClientDIAT,
+    llvm::Pass &P, llvm::Function &F)
+  : DIClientServerInfo(P, F), ClientDIAT(&ClientDIAT) {}
 
 bcl::tagged_pair<
   bcl::tagged<DIMemory *, Origin>, bcl::tagged<DIMemory *, Clone>>