Refactor inlining of recognized methods.

Refactor inlining of recognized methods.

Move some common code into the FlowGraphInliner to avoid duplication.

Also, only emit the receiver class check when necessary in the first place.

There is more duplicate code in jit_optimizer.cc and aot_optimizer.cc which is
independent of the compilation mode. This CL is just a step.

Removed receiver class check for SIMD operations that require unboxing: The unbox operation already checks the argument type.

BUG=
R=vegorov@google.com

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

[Florian Schneider, 2016-06-27 20:45:20]

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

[Florian Schneider, 2016-06-27 22:12:21]

Stand by - I'm investigating a performance regression caused by this.

[Florian Schneider, 2016-06-28 00:03:06]

Description was changed from

==========
Refactor inlining of recognized methods.

Move some common code into the FlowGraphInliner to avoid duplication.

Also, only emit the receiver class check when necessary in the first place.

There is more duplicate code in jit_optimizer.cc and aot_optimizer.cc which is
independent of the compilation mode. This CL is just a step.

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

to

==========
Refactor inlining of recognized methods.

Move some common code into the FlowGraphInliner to avoid duplication.

Also, only emit the receiver class check when necessary in the first place.

There is more duplicate code in jit_optimizer.cc and aot_optimizer.cc which is
independent of the compilation mode. This CL is just a step.

Removed receiver class check for SIMD operations that require unboxing: The
unbox operation already checks the argument type.

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

[Florian Schneider, 2016-06-28 01:56:48]

PTAL

[Vyacheslav Egorov (Google), 2016-06-28 12:31:58]

lgtm

[Florian Schneider, 2016-06-28 18:25:26]

Description was changed from

==========
Refactor inlining of recognized methods.

Move some common code into the FlowGraphInliner to avoid duplication.

Also, only emit the receiver class check when necessary in the first place.

There is more duplicate code in jit_optimizer.cc and aot_optimizer.cc which is
independent of the compilation mode. This CL is just a step.

Removed receiver class check for SIMD operations that require unboxing: The
unbox operation already checks the argument type.

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

to

==========
Refactor inlining of recognized methods.

Move some common code into the FlowGraphInliner to avoid duplication.

Also, only emit the receiver class check when necessary in the first place.

There is more duplicate code in jit_optimizer.cc and aot_optimizer.cc which is
independent of the compilation mode. This CL is just a step.

Removed receiver class check for SIMD operations that require unboxing: The
unbox operation already checks the argument type.

BUG=
R=vegorov@google.com

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

[Florian Schneider, 2016-06-28 18:25:28]

Committed patchset #2 (id:20001) manually as
a3dcdb35aa269447ca39c809aa6a848a725f1c66 (presubmit successful).