Commit 1d3f32ea authored by ishell@chromium.org's avatar ishell@chromium.org

Don't disable hash randomization option in predictable mode.

We don't need it because
1) hash randomization uses random generator whose results depend on --random-seed option which is set in predictable mode.
2) disabling hash randomization triggers assert in snapshot mode.

R=svenpanne@chromium.org

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

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@19647 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 766af5c2
...@@ -653,7 +653,6 @@ DEFINE_bool(profile_hydrogen_code_stub_compilation, false, ...@@ -653,7 +653,6 @@ DEFINE_bool(profile_hydrogen_code_stub_compilation, false,
"Print the time it takes to lazily compile hydrogen code stubs.") "Print the time it takes to lazily compile hydrogen code stubs.")
DEFINE_bool(predictable, false, "enable predictable mode") DEFINE_bool(predictable, false, "enable predictable mode")
DEFINE_neg_implication(predictable, randomize_hashes)
DEFINE_neg_implication(predictable, concurrent_recompilation) DEFINE_neg_implication(predictable, concurrent_recompilation)
DEFINE_neg_implication(predictable, concurrent_osr) DEFINE_neg_implication(predictable, concurrent_osr)
DEFINE_neg_implication(predictable, concurrent_sweeping) DEFINE_neg_implication(predictable, concurrent_sweeping)
......
...@@ -187,13 +187,10 @@ void V8::RunMicrotasks(Isolate* isolate) { ...@@ -187,13 +187,10 @@ void V8::RunMicrotasks(Isolate* isolate) {
void V8::InitializeOncePerProcessImpl() { void V8::InitializeOncePerProcessImpl() {
FlagList::EnforceFlagImplications(); FlagList::EnforceFlagImplications();
if (FLAG_predictable) { if (FLAG_predictable && FLAG_random_seed == 0) {
if (FLAG_random_seed == 0) {
// Avoid random seeds in predictable mode. // Avoid random seeds in predictable mode.
FLAG_random_seed = 12347; FLAG_random_seed = 12347;
} }
FLAG_hash_seed = 0;
}
if (FLAG_stress_compaction) { if (FLAG_stress_compaction) {
FLAG_force_marking_deque_overflows = true; FLAG_force_marking_deque_overflows = true;
......
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