Tweak Florian's dart2js inlining heuristic change

Tweak Florian's dart2js inlining heuristic change

This CL updates https://chromiumcodereview.appspot.com/2558633007/

R=floitsch@google.com

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

[sra1, 2017-02-28 22:54:59]

Patchset #1 (id:1) has been deleted

[sra1, 2017-02-28 23:09:52]

Patchset #1 (id:20001) has been deleted

[sra1, 2017-02-28 23:18:05]

Patchset #1 (id:40001) has been deleted

[floitsch, 2017-03-01 08:50:15]

LGTM.

https://codereview.chromium.org/2724473004/diff/60001/pkg/compiler/lib/src/ss...
File pkg/compiler/lib/src/ssa/builder.dart (right):

https://codereview.chromium.org/2724473004/diff/60001/pkg/compiler/lib/src/ss...
pkg/compiler/lib/src/ssa/builder.dart:6686: // parameter is a constant. Assuming
the parameter is constant, we can
In theory we could even look at the parameter. However, then we would need to
change our caching strategy for "isInlinable".

[sra1, 2017-03-07 04:33:06]

Description was changed from

==========
Tweak Florian's dart2js inlining heuristic change

This CL updates https://chromiumcodereview.appspot.com/2558633007/

R=floitsch@google.com
==========

to

==========
Tweak Florian's dart2js inlining heuristic change

This CL updates https://chromiumcodereview.appspot.com/2558633007/

R=floitsch@google.com

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

[sra1, 2017-03-07 04:33:08]

Committed patchset #1 (id:60001) manually as
bf9a4502c889d6542e5e1e7b0b72c67c81418336 (presubmit successful).