Commit d81edba7 authored by rmcilroy's avatar rmcilroy Committed by Commit bot

[Interpreter] Set bytecode array in Runtime_SetCode.

Set the bytecode array correctly in Runtime_SetCode.
This fixes issues with building the snapshot with ignition enabled.

BUG=v8:4280
LOG=N

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

Cr-Commit-Position: refs/heads/master@{#33638}
parent ca6587c0
...@@ -162,6 +162,9 @@ RUNTIME_FUNCTION(Runtime_SetCode) { ...@@ -162,6 +162,9 @@ RUNTIME_FUNCTION(Runtime_SetCode) {
// Set the code, scope info, formal parameter count, and the length // Set the code, scope info, formal parameter count, and the length
// of the target shared function info. // of the target shared function info.
target_shared->ReplaceCode(source_shared->code()); target_shared->ReplaceCode(source_shared->code());
if (source_shared->HasBytecodeArray()) {
target_shared->set_function_data(source_shared->bytecode_array());
}
target_shared->set_scope_info(source_shared->scope_info()); target_shared->set_scope_info(source_shared->scope_info());
target_shared->set_length(source_shared->length()); target_shared->set_length(source_shared->length());
target_shared->set_feedback_vector(source_shared->feedback_vector()); target_shared->set_feedback_vector(source_shared->feedback_vector());
......
...@@ -508,7 +508,6 @@ ...@@ -508,7 +508,6 @@
'test-debug/*' : [SKIP], 'test-debug/*' : [SKIP],
'test-func-name-inference/*' : [SKIP], 'test-func-name-inference/*' : [SKIP],
'test-inobject-slack-tracking/*' : [SKIP], 'test-inobject-slack-tracking/*' : [SKIP],
'test-serialize/*' : [SKIP],
# TODO(mstarzinger,4674): Message object is not properly cleared. # TODO(mstarzinger,4674): Message object is not properly cleared.
'test-heap/MessageObjectLeak': [FAIL], 'test-heap/MessageObjectLeak': [FAIL],
...@@ -519,6 +518,12 @@ ...@@ -519,6 +518,12 @@
# TODO(mstarzinger,4674): Bug in control flow modeling. Investigate and fix. # TODO(mstarzinger,4674): Bug in control flow modeling. Investigate and fix.
'test-run-jsexceptions/ThrowMessageIndirectly': [FAIL], 'test-run-jsexceptions/ThrowMessageIndirectly': [FAIL],
# TODO(rmcilroy,4680): Check failure in runtime-compiler.cc on code type.
'test-serialize/SerializeInternalReference': [FAIL],
# TODO(rmcilroy,4680): Check failed: toplevel_test_code_event_found.
'test-serialize/SerializeToplevelIsolates': [FAIL],
'test-api-interceptors/InterceptorCallICInvalidatedConstantFunctionViaGlobal': [SKIP], 'test-api-interceptors/InterceptorCallICInvalidatedConstantFunctionViaGlobal': [SKIP],
'test-api-interceptors/InterceptorLoadICInvalidatedCallbackViaGlobal': [SKIP], 'test-api-interceptors/InterceptorLoadICInvalidatedCallbackViaGlobal': [SKIP],
'test-api-interceptors/InterceptorLoadICInvalidatedFieldViaGlobal': [SKIP], 'test-api-interceptors/InterceptorLoadICInvalidatedFieldViaGlobal': [SKIP],
...@@ -537,7 +542,6 @@ ...@@ -537,7 +542,6 @@
'test-heap/IncrementalMarkingPreservesMonomorphicCallIC': [SKIP], 'test-heap/IncrementalMarkingPreservesMonomorphicCallIC': [SKIP],
'test-heap/IncrementalMarkingPreservesMonomorphicConstructor': [SKIP], 'test-heap/IncrementalMarkingPreservesMonomorphicConstructor': [SKIP],
'test-heap/NoWeakHashTableLeakWithIncrementalMarking': [SKIP], 'test-heap/NoWeakHashTableLeakWithIncrementalMarking': [SKIP],
'test-heap-profiler/HeapSnapshotSimd': [SKIP],
'test-heap/OptimizedAllocationAlwaysInNewSpace': [SKIP], 'test-heap/OptimizedAllocationAlwaysInNewSpace': [SKIP],
'test-heap/Regress169209': [SKIP], 'test-heap/Regress169209': [SKIP],
'test-heap/Regress357137': [SKIP], 'test-heap/Regress357137': [SKIP],
...@@ -584,9 +588,14 @@ ...@@ -584,9 +588,14 @@
'test-unscopables-hidden-prototype/Unscopables': [SKIP], 'test-unscopables-hidden-prototype/Unscopables': [SKIP],
}], # ignition == True }], # ignition == True
['ignition == True and arch == arm', {
'test-serialize/SerializeInternalReference': [SKIP],
}],
['ignition == True and arch == arm64', { ['ignition == True and arch == arm64', {
'test-decls/CrossScriptReferencesHarmony': [SKIP], 'test-decls/CrossScriptReferencesHarmony': [SKIP],
'test-js-arm64-variables/lookup_slots': [SKIP], 'test-js-arm64-variables/lookup_slots': [SKIP],
'test-serialize/SerializeInternalReference': [SKIP],
'test-spaces/SizeOfFirstPageIsLargeEnough': [SKIP], 'test-spaces/SizeOfFirstPageIsLargeEnough': [SKIP],
'test-heap/AddInstructionChangesNewSpacePromotion': [SKIP], 'test-heap/AddInstructionChangesNewSpacePromotion': [SKIP],
'test-heap/Regress538257': [SKIP], 'test-heap/Regress538257': [SKIP],
......
...@@ -797,7 +797,6 @@ ...@@ -797,7 +797,6 @@
'declare-locally': [SKIP], 'declare-locally': [SKIP],
'delete-in-with': [SKIP], 'delete-in-with': [SKIP],
'deserialize-optimize-inner': [SKIP], 'deserialize-optimize-inner': [SKIP],
'double-equals': [SKIP],
'eval-enclosing-function-name': [SKIP], 'eval-enclosing-function-name': [SKIP],
'eval': [SKIP], 'eval': [SKIP],
'eval-stack-trace': [SKIP], 'eval-stack-trace': [SKIP],
......
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