Pack deopt ID into the return before and return after deopt instructions.

Pack deopt ID into the return before and return after deopt instructions.

Pack it into a machine word-sized bit field along with the object table
index.  The commands do not implicitly use the object table index any more,
which allows removing deopt IDs from the object table and pooling function
entries.

R=srdjan@google.com
BUG=

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

[srdjan, 2012-10-02 17:34:49]

https://codereview.chromium.org/11014020/diff/1/runtime/vm/deopt_instructions.cc
File runtime/vm/deopt_instructions.cc (right):

https://codereview.chromium.org/11014020/diff/1/runtime/vm/deopt_instructions...
runtime/vm/deopt_instructions.cc:140: }
How about overloading constructor with:
DeoptRetAddrAfterInstr(intptr_t from_index), thus the knowledge of encoding of
from_index is hidden inside the class.

https://codereview.chromium.org/11014020/diff/1/runtime/vm/deopt_instructions...
runtime/vm/deopt_instructions.cc:186: : object_table_index_(object_table_index),
deopt_id_(deopt_id) {
ditto.

[Kevin Millikin (Google), 2012-10-03 07:05:36]

https://codereview.chromium.org/11014020/diff/1/runtime/vm/deopt_instructions.cc
File runtime/vm/deopt_instructions.cc (right):

https://codereview.chromium.org/11014020/diff/1/runtime/vm/deopt_instructions...
runtime/vm/deopt_instructions.cc:140: }
On 2012/10/02 17:34:49, srdjan wrote:
> How about overloading constructor with:
> DeoptRetAddrAfterInstr(intptr_t from_index), thus the knowledge of encoding of
> from_index is hidden inside the class.

OK.

[Kevin Millikin (Google), 2012-10-03 14:10:43]

Comments addressed.  Please take another look.

[srdjan, 2012-10-03 15:33:51]

lgtm