Commit e37e6fc7 authored by mstarzinger's avatar mstarzinger Committed by Commit bot

[compiler] Make --hydrogen-filter only apply to Crankshaft.

This makes sure that --hydrogen-filter only filters for Crankshaft, not
for TurboFan compilations. For TurboFan there is --turbo-filter as a
separate flag already. There no longer is a single flag to filter both
compilers at the same time, one can still specify both flags however.

R=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/1871343003

Cr-Commit-Position: refs/heads/master@{#35388}
parent 086bc498
...@@ -326,11 +326,6 @@ OptimizedCompileJob::Status OptimizedCompileJob::CreateGraph() { ...@@ -326,11 +326,6 @@ OptimizedCompileJob::Status OptimizedCompileJob::CreateGraph() {
return AbortOptimization(kOptimizedTooManyTimes); return AbortOptimization(kOptimizedTooManyTimes);
} }
// Check the whitelist for Crankshaft.
if (!info()->shared_info()->PassesFilter(FLAG_hydrogen_filter)) {
return AbortOptimization(kHydrogenFilter);
}
// Optimization requires a version of fullcode with deoptimization support. // Optimization requires a version of fullcode with deoptimization support.
// Recompile the unoptimized version of the code if the current version // Recompile the unoptimized version of the code if the current version
// doesn't have deoptimization support already. // doesn't have deoptimization support already.
......
...@@ -119,6 +119,11 @@ HCompilationJob::Status HCompilationJob::CreateGraphImpl() { ...@@ -119,6 +119,11 @@ HCompilationJob::Status HCompilationJob::CreateGraphImpl() {
return FAILED; return FAILED;
} }
// Check the whitelist for Crankshaft.
if (!info()->shared_info()->PassesFilter(FLAG_hydrogen_filter)) {
return AbortOptimization(kHydrogenFilter);
}
Scope* scope = info()->scope(); Scope* scope = info()->scope();
if (LUnallocated::TooManyParameters(scope->num_parameters())) { if (LUnallocated::TooManyParameters(scope->num_parameters())) {
// Crankshaft would require too many Lithium operands. // Crankshaft would require too many Lithium operands.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment