[Attributor][FIX] Undo 16188f9 until SCC iterator bug is fixed
The buildbot
http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win
shows some strange SCC iterator bug since 16188f9 which we need to
investigate. This patch should remove the part of 16188f9 that could
have exposed the problem.
diff --git a/llvm/lib/Transforms/IPO/Attributor.cpp b/llvm/lib/Transforms/IPO/Attributor.cpp
index 13a5191..429123a 100644
--- a/llvm/lib/Transforms/IPO/Attributor.cpp
+++ b/llvm/lib/Transforms/IPO/Attributor.cpp
@@ -4627,9 +4627,7 @@
Value &V = getAssociatedValue();
if (V.getType()->isPointerTy() &&
V.getType()->getPointerElementType()->isFunctionTy() &&
- !A.isModulePass() &&
- A.getInfoCache().getAnalysisResultForFunction<LoopAnalysis>(
- *getAnchorScope()))
+ !A.isModulePass())
indicatePessimisticFixpoint();
}
@@ -4756,7 +4754,9 @@
/// See AbstractAttribute::initialize(...).
void initialize(Attributor &A) override {
- AAValueSimplifyImpl::initialize(A);
+ // FIXME: This might have exposed a SCC iterator update bug in the old PM.
+ // Needs investigation.
+ // AAValueSimplifyImpl::initialize(A);
Value &V = getAnchorValue();
// TODO: add other stuffs