Eliminate object table and use regular object pool for deoptimization infos.

Eliminate object table and use regular object pool for deoptimization infos.

On platforms that already use an object pool, we had a separate object table
used only for deoptimization infos. With this CL we don't have separate
tables, but only use the common object pool for both purposes.

R=vegorov@google.com

Committed: https://code.google.com/p/dart/source/detail?r=45108

[Florian Schneider, 2015-04-13 12:12:48]

fschneider@google.com changed reviewers:
+ vegorov@google.com

[Vyacheslav Egorov (Google), 2015-04-13 15:31:37]

LGTM

https://codereview.chromium.org/1073173003/diff/20001/runtime/vm/compiler.cc
File runtime/vm/compiler.cc (right):

https://codereview.chromium.org/1073173003/diff/20001/runtime/vm/compiler.cc#...
runtime/vm/compiler.cc:723: graph_compiler.FinalizeDeoptInfo(&assembler));
Maybe add a comment that FinalizeDeoptInfo populates the pool so it has to be
called before the FinalizeCode.

Also other FinalizeXYZ methods take code and change it - so maybe rename
FinalizeDeoptInfo to CreateDeoptInfo()

[Florian Schneider, 2015-04-13 15:37:27]

https://codereview.chromium.org/1073173003/diff/20001/runtime/vm/compiler.cc
File runtime/vm/compiler.cc (right):

https://codereview.chromium.org/1073173003/diff/20001/runtime/vm/compiler.cc#...
runtime/vm/compiler.cc:723: graph_compiler.FinalizeDeoptInfo(&assembler));
On 2015/04/13 15:31:37, Vyacheslav Egorov (Google) wrote:
> Maybe add a comment that FinalizeDeoptInfo populates the pool so it has to be
> called before the FinalizeCode.
> 
> Also other FinalizeXYZ methods take code and change it - so maybe rename
> FinalizeDeoptInfo to CreateDeoptInfo()

Done.

[Florian Schneider, 2015-04-13 15:44:16]

Committed patchset #4 (id:60001) manually as r45108 (presubmit successful).