[llvm] r271159 - [PM] Reassociate: cache analyses more aggressively.

[Davide Italiano via llvm-commits]

Author: davide
Date: Sat May 28 19:41:17 2016
New Revision: 271159

URL: http://llvm.org/viewvc/llvm-project?rev=271159&view=rev
Log:
[PM] Reassociate: cache analyses more aggressively.

While here, add a FIXME for setPreserveCFG().

Modified:
    llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp?rev=271159&r1=271158&r2=271159&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp Sat May 28 19:41:17 2016
@@ -2227,8 +2227,13 @@ PreservedAnalyses ReassociatePass::run(F
   RankMap.clear();
   ValueRankMap.clear();
 
-  if (MadeChange)
-    return PreservedAnalyses::none();
+  if (MadeChange) {
+    // FIXME: Reassociate should also 'preserve the CFG'.
+    // The new pass manager has currently no way to do it.
+    auto PA = PreservedAnalyses();
+    PA.preserve<GlobalsAA>();
+    return PA;
+  }
 
   return PreservedAnalyses::all();
 }