Simplify and improve optimization of is-tests in the precompiler.

Simplify and improve optimization of is-tests in the precompiler.

The optimizer now also deals with is-checks for num, double, _Smi in the
same way as it did for int before.

This CL eliminates many is-tests of constants that appear after inlining.

Also, make a better distinction between deoptimizing and non-deoptimizing
cases when replacing is-tests. e.g. InstanceOfInstr can't cause eager
deoptimization.

Also, improve tracing output when running e.g. --trace-inlining

Minor fix in raw_object.h.

BUG=
R=vegorov@google.com

Committed: https://github.com/dart-lang/sdk/commit/220b040c4a88086dc4125fd2cfbe610d2db3dac9

[Florian Schneider, 2016-06-15 23:26:08]

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

[Florian Schneider, 2016-06-16 00:57:07]

This also fixes a performance regression introduced in
https://github.com/dart-lang/sdk/commit/9ee5dfb2451e294555f8b3320a1580f9bc315859

[Vyacheslav Egorov (Google), 2016-06-16 08:37:38]

lgtm

https://codereview.chromium.org/2074533002/diff/1/runtime/vm/raw_object.h
File runtime/vm/raw_object.h (left):

https://codereview.chromium.org/2074533002/diff/1/runtime/vm/raw_object.h#old...
runtime/vm/raw_object.h:2284: return (index == kOneByteStringCid ||
I wonder how this worked

[Florian Schneider, 2016-06-16 15:54:34]

Description was changed from

==========
Simplify and improve optimization of is-tests in the precompiler.

The optimizer now also deals with is-checks for num, double, _Smi in the
same way as it did for int before.

This CL eliminates many is-tests of constants that appear after inlining.

Also, make a better distinction between deoptimizing and non-deoptimizing
cases when replacing is-tests. e.g. InstanceOfInstr can't cause eager
deoptimization.

Also, improve tracing output when running e.g. --trace-inlining

Minor fix in raw_object.h.

BUG=
==========

to

==========
Simplify and improve optimization of is-tests in the precompiler.

The optimizer now also deals with is-checks for num, double, _Smi in the
same way as it did for int before.

This CL eliminates many is-tests of constants that appear after inlining.

Also, make a better distinction between deoptimizing and non-deoptimizing
cases when replacing is-tests. e.g. InstanceOfInstr can't cause eager
deoptimization.

Also, improve tracing output when running e.g. --trace-inlining

Minor fix in raw_object.h.

BUG=
R=vegorov@google.com

Committed:
https://github.com/dart-lang/sdk/commit/220b040c4a88086dc4125fd2cfbe610d2db3dac9
==========

[Florian Schneider, 2016-06-16 15:54:36]

Committed patchset #1 (id:1) manually as
220b040c4a88086dc4125fd2cfbe610d2db3dac9 (presubmit successful).