Commit cb433bed authored by mtrofin's avatar mtrofin Committed by Commit bot

[wasm] simpler detection if we compiled asm-wasm

BUG=643595

Review-Url: https://codereview.chromium.org/2582583002
Cr-Commit-Position: refs/heads/master@{#41738}
parent 03f33f2e
...@@ -1366,22 +1366,12 @@ bool CodeGenerationFromStringsAllowed(Isolate* isolate, ...@@ -1366,22 +1366,12 @@ bool CodeGenerationFromStringsAllowed(Isolate* isolate,
} }
} }
bool ContainsAsmModule(const Scope* scope, Zone* zone) { bool ContainsAsmModule(Handle<Script> script) {
DCHECK_NOT_NULL(scope); WeakFixedArray::Iterator iter(script->shared_function_infos());
DCHECK_NOT_NULL(zone); DisallowHeapAllocation no_gc;
ZoneQueue<const Scope*> worklist(zone);
// We assume scopes form a tree, so no need to check for cycles while (SharedFunctionInfo* info = iter.Next<SharedFunctionInfo>()) {
worklist.push(scope); if (info->HasAsmWasmData()) return true;
while (!worklist.empty()) {
const Scope* s = worklist.front();
worklist.pop();
if (s->IsAsmModule()) {
return true;
}
for (const Scope* child = s->inner_scope(); child != nullptr;
child = child->sibling()) {
worklist.push(child);
}
} }
return false; return false;
} }
...@@ -1528,7 +1518,7 @@ Handle<SharedFunctionInfo> Compiler::GetSharedFunctionInfoForScript( ...@@ -1528,7 +1518,7 @@ Handle<SharedFunctionInfo> Compiler::GetSharedFunctionInfoForScript(
compilation_cache->PutScript(source, context, language_mode, result); compilation_cache->PutScript(source, context, language_mode, result);
if (FLAG_serialize_toplevel && if (FLAG_serialize_toplevel &&
compile_options == ScriptCompiler::kProduceCodeCache && compile_options == ScriptCompiler::kProduceCodeCache &&
!ContainsAsmModule(info.scope(), &zone)) { !ContainsAsmModule(script)) {
HistogramTimerScope histogram_timer( HistogramTimerScope histogram_timer(
isolate->counters()->compile_serialize()); isolate->counters()->compile_serialize());
RuntimeCallTimerScope runtimeTimer(isolate, RuntimeCallTimerScope runtimeTimer(isolate,
......
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