diff options
author | Philip Reames <listmail@philipreames.com> | 2015-02-20 02:34:49 +0000 |
---|---|---|
committer | Philip Reames <listmail@philipreames.com> | 2015-02-20 02:34:49 +0000 |
commit | 6faacf47721848be8eae706ff9f177b8b7c0b527 (patch) | |
tree | 427d2a14f733e4df75a075ab2933318d72596c9a /llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp | |
parent | f86505abdf501b257271e718742ee76b96ca969c (diff) | |
download | bcm5719-llvm-6faacf47721848be8eae706ff9f177b8b7c0b527.tar.gz bcm5719-llvm-6faacf47721848be8eae706ff9f177b8b7c0b527.zip |
Adjust enablement of RewriteStatepointsForGC
When back merging the changes in 229945 I noticed that I forgot to mark the test cases with the appropriate GC. We want the rewriting to be off by default (even when manually added to the pass order), not on-by default. To keep the current test working, mark them as using the statepoint-example GC and whitelist that GC.
Longer term, we need a better selection mechanism here for both actual usage and testing. As I migrate more tests to the in tree version of this pass, I will probably need to update the enable/disable logic as well.
llvm-svn: 229954
Diffstat (limited to 'llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp')
-rw-r--r-- | llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp b/llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp index 71c79309df8..2fa5d0cef29 100644 --- a/llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp +++ b/llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp @@ -1900,7 +1900,8 @@ static bool insertParsePoints(Function &F, DominatorTree &DT, Pass *P, /// point of this function is as an extension point for custom logic. static bool shouldRewriteStatepointsIn(Function &F) { // TODO: This should check the GCStrategy - return true; + const std::string StatepointExampleName("statepoint-example"); + return StatepointExampleName == F.getGC(); } bool RewriteStatepointsForGC::runOnFunction(Function &F) { |