Compress memory usage. Imprive speed of finding exception handlers: if a code does not have any han…

Compress memory usage. Imprive speed of finding exception handlers: if a code does not have any handlers, do not reaverse the PC descriptors.

R=asiva@google.com

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

[siva, 2013-08-29 16:29:20]

lgtm

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/exceptions.cc
File runtime/vm/exceptions.cc (right):

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/exceptions.cc#n...
runtime/vm/exceptions.cc:369: // 'error_class' is not a predefined class.
// 'class Error' is not a predefined class.

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/object.h
File runtime/vm/object.h (right):

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/object.h#newcod...
runtime/vm/object.h:2947: void set_state_bits(intptr_t bits) const;
Does this have to be a public method?

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/raw_object.h
File runtime/vm/raw_object.h (right):

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/raw_object.h#ne...
runtime/vm/raw_object.h:841: // Thes field cannot be shorter because of
alignment issues on x64
*Thes*This

[srdjan, 2013-08-29 16:38:12]

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/exceptions.cc
File runtime/vm/exceptions.cc (right):

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/exceptions.cc#n...
runtime/vm/exceptions.cc:369: // 'error_class' is not a predefined class.
On 2013/08/29 16:29:20, siva wrote:
> // 'class Error' is not a predefined class.

Done.

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/object.h
File runtime/vm/object.h (right):

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/object.h#newcod...
runtime/vm/object.h:2947: void set_state_bits(intptr_t bits) const;
On 2013/08/29 16:29:20, siva wrote:
> Does this have to be a public method?

Made private.

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/raw_object.h
File runtime/vm/raw_object.h (right):

https://codereview.chromium.org/23691008/diff/8001/runtime/vm/raw_object.h#ne...
runtime/vm/raw_object.h:841: // Thes field cannot be shorter because of
alignment issues on x64
On 2013/08/29 16:29:20, siva wrote:
> *Thes*This

Done.

[srdjan, 2013-08-29 16:47:54]

Committed patchset #3 manually as r26880 (presubmit successful).