https://bugs.llvm.org/show_bug.cgi?id=45933
Bug ID: 45933
Summary: [AssumeBundles] Missing AU.addRequired() call
Product: libraries
Version: 10.0
Hardware: PC
OS: Linux
Status: NEW
Severity: enhancement
Priority: P
Component: Transformation Utilities
Assignee: unassignedb...@nondot.org
Reporter: pauls...@linux.vnet.ibm.com
CC: llvm-bugs@lists.llvm.org
It seems that the AssumeSimplify pass calls
getAnalysis<AssumptionCacheTracker>() without requiring it, and I get:
clang ~/tmp/test.c -save-temps -c -S -O3 -mllvm -enable-knowledge-retention
PassAnalysisSupport.h:236:
Assertion `ResultPass && "getAnalysis*() called on an analysis that was not "
"'required' by pass!"' failed
Is the following one-line patch good?
diff --git a/llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp
b/llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp
index ee5e142..631661a 100644
--- a/llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp
+++ b/llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp
@@ -467,6 +467,7 @@ public:
void getAnalysisUsage(AnalysisUsage &AU) const override {
AU.setPreservesAll();
+ AU.addRequired<AssumptionCacheTracker>();
}
};
--
You are receiving this mail because:
You are on the CC list for the bug._______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
