RPCSX/llvm
1
0
Fork 0
mirror of https://github.com/RPCSX/llvm.git synced 2025-01-25 05:34:59 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[PM] Normalize FIXMEs for missing PreserveCFG to have the same wording.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@273974 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Michael Kuperstein 2016-06-28 00:54:12 +00:00
parent 5dcd187721
commit ce2862caac
9 changed files with 9 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/Transforms/InstCombine/InstructionCombining.cpp
View File

@ -3111,7 +3111,7 @@ PreservedAnalyses InstCombinePass::run(Function &F,
return PreservedAnalyses::all(); return PreservedAnalyses::all();
// Mark all the analyses that instcombine updates as preserved. // Mark all the analyses that instcombine updates as preserved.
// FIXME: Need a way to preserve CFG analyses here! // FIXME: This should also 'preserve the CFG'.
PreservedAnalyses PA; PreservedAnalyses PA;
PA.preserve<DominatorTreeAnalysis>(); PA.preserve<DominatorTreeAnalysis>();
return PA; return PA;

3
lib/Transforms/Scalar/ADCE.cpp
View File

@ -149,8 +149,7 @@ PreservedAnalyses ADCEPass::run(Function &F, FunctionAnalysisManager &) {
if (!aggressiveDCE(F)) if (!aggressiveDCE(F))
return PreservedAnalyses::all(); return PreservedAnalyses::all();
// FIXME: ADCE should also 'preserve the CFG'. // FIXME: This should also 'preserve the CFG'.
// The new pass manager has currently no way to do it.
auto PA = PreservedAnalyses(); auto PA = PreservedAnalyses();
PA.preserve<GlobalsAA>(); PA.preserve<GlobalsAA>();
return PA; return PA;

3
lib/Transforms/Scalar/BDCE.cpp
View File

@ -74,8 +74,7 @@ PreservedAnalyses BDCEPass::run(Function &F, FunctionAnalysisManager &AM) {
if (!bitTrackingDCE(F, DB)) if (!bitTrackingDCE(F, DB))
return PreservedAnalyses::all(); return PreservedAnalyses::all();
// FIXME: BDCE should also 'preserve the CFG'. // FIXME: This should also 'preserve the CFG'.
// The new pass manager has currently no way to do it.
auto PA = PreservedAnalyses(); auto PA = PreservedAnalyses();
PA.preserve<GlobalsAA>(); PA.preserve<GlobalsAA>();
return PA; return PA;

2
lib/Transforms/Scalar/Float2Int.cpp
View File

@ -532,7 +532,7 @@ PreservedAnalyses Float2IntPass::run(Function &F, FunctionAnalysisManager &) {
if (!runImpl(F)) if (!runImpl(F))
return PreservedAnalyses::all(); return PreservedAnalyses::all();
else { else {
//FIXME: setPreservesCFG is not currently supported in the new PM. // FIXME: This should also 'preserve the CFG'.
PreservedAnalyses PA; PreservedAnalyses PA;
PA.preserve<GlobalsAA>(); PA.preserve<GlobalsAA>();
return PA; return PA;

3
lib/Transforms/Scalar/IndVarSimplify.cpp
View File

@ -2233,8 +2233,7 @@ PreservedAnalyses IndVarSimplifyPass::run(Loop &L, AnalysisManager<Loop> &AM) {
if (!IVS.run(&L)) if (!IVS.run(&L))
return PreservedAnalyses::all(); return PreservedAnalyses::all();
// FIXME: once we have an equivalent of AU.setPreservesCFG() in the // FIXME: This should also 'preserve the CFG'.
// new pass manager, we should use that here.
return getLoopPassPreservedAnalyses(); return getLoopPassPreservedAnalyses();
} }

3
lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
View File

@ -595,8 +595,7 @@ MergedLoadStoreMotionPass::run(Function &F, AnalysisManager<Function> &AM) {
if (!Impl.run(F, MD, AA)) if (!Impl.run(F, MD, AA))
return PreservedAnalyses::all(); return PreservedAnalyses::all();
// FIXME: This pass should also 'preserve the CFG'. // FIXME: This should also 'preserve the CFG'.
// The new pass manager has currently no way to do it.
PreservedAnalyses PA; PreservedAnalyses PA;
PA.preserve<GlobalsAA>(); PA.preserve<GlobalsAA>();
PA.preserve<MemoryDependenceAnalysis>(); PA.preserve<MemoryDependenceAnalysis>();

3
lib/Transforms/Scalar/Reassociate.cpp
View File

@ -2226,8 +2226,7 @@ PreservedAnalyses ReassociatePass::run(Function &F, FunctionAnalysisManager &) {
ValueRankMap.clear(); ValueRankMap.clear();
if (MadeChange) { if (MadeChange) {
// FIXME: Reassociate should also 'preserve the CFG'. // FIXME: This should also 'preserve the CFG'.
// The new pass manager has currently no way to do it.
auto PA = PreservedAnalyses(); auto PA = PreservedAnalyses();
PA.preserve<GlobalsAA>(); PA.preserve<GlobalsAA>();
return PA; return PA;

3
lib/Transforms/Utils/LCSSA.cpp
View File

@ -340,8 +340,7 @@ PreservedAnalyses LCSSAPass::run(Function &F, AnalysisManager<Function> &AM) {
if (!formLCSSAOnAllLoops(&LI, DT, SE)) if (!formLCSSAOnAllLoops(&LI, DT, SE))
return PreservedAnalyses::all(); return PreservedAnalyses::all();
// FIXME: There is no setPreservesCFG in the new PM. When that becomes // FIXME: This should also 'preserve the CFG'.
// available, it should be used here.
PreservedAnalyses PA; PreservedAnalyses PA;
PA.preserve<BasicAA>(); PA.preserve<BasicAA>();
PA.preserve<GlobalsAA>(); PA.preserve<GlobalsAA>();

6
lib/Transforms/Utils/Mem2Reg.cpp
View File

@ -59,11 +59,7 @@ PreservedAnalyses PromotePass::run(Function &F, AnalysisManager<Function> &AM) {
if (!promoteMemoryToRegister(F, DT, AC)) if (!promoteMemoryToRegister(F, DT, AC))
return PreservedAnalyses::all(); return PreservedAnalyses::all();
// FIXME: This pass should preserve the CFG. // FIXME: This should also 'preserve the CFG'.
// There's currently no way to do it in the new PM.
// In the old PM this pass preserved a fair amount of "orthogonal"
// transformation passes. This concept has no sense in the new PM,
// therefore we don't preserve them here.
return PreservedAnalyses::none(); return PreservedAnalyses::none();
} }