lgtm
http://codereview.chromium.org/9325019/diff/1/src/arm/lithium-arm.cc File src/arm/lithium-arm.cc (right): http://codereview.chromium.org/9325019/diff/1/src/arm/lithium-arm.cc#newcode795 src/arm/lithium-arm.cc:795: if (!allocator_->AllocationOk()) Abort("Not enough virtual registers."); On 2012/02/03 09:17:41, fschneider wrote:
Abort in this case is doing exactly that, namely bailing out of the
lithium
construction by setting the is_aborted flag which is checked in LChunkBuilder::DoBasicBlock.
Ooops, I've missed the Abort method in LChunkBuilder, sorry... %-} http://codereview.chromium.org/9325019/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
