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

[turbofan] Moved split edge form validation lower.

There are multiple ways we may construct an instruction sequence.
Moved the split edge form validation where they must all eventually
chokepoint to - the register allocation pipeline, which is also the main
and first consumer of this property.

BUG=

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

Cr-Commit-Position: refs/heads/master@{#34903}
parent 2701e67d
......@@ -682,10 +682,6 @@ InstructionSequence::InstructionSequence(Isolate* isolate,
representations_(zone()),
deoptimization_entries_(zone()) {
block_starts_.reserve(instruction_blocks_->size());
#if DEBUG
ValidateEdgeSplitForm();
#endif
}
......
......@@ -1445,6 +1445,7 @@ void Pipeline::AllocateRegisters(const RegisterConfiguration* config,
base::SmartArrayPointer<char> debug_name;
#ifdef DEBUG
debug_name = info()->GetDebugName();
data_->sequence()->ValidateEdgeSplitForm();
#endif
data->InitializeRegisterAllocationData(config, descriptor, debug_name.get());
......
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