dart2js ssa: Inlining heuristics ignore assert() statement size.

dart2js ssa: Inlining heuristics ignore assert() statement size.

assert() Ast nodes are not counted towards inlinee size in production mode.

R=sigmund@google.com

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

[sra1, 2015-09-17 03:42:53]

Patchset #1 (id:1) has been deleted

[sra1, 2015-09-17 03:43:58]

sra@google.com changed reviewers:
+ sigmund@google.com

[sra1, 2015-09-17 03:43:59]

I didn't see any regressions from the @NoInline()

[Siggi Cherem (dart-lang), 2015-09-17 16:10:10]

lgtm

https://codereview.chromium.org/1345413002/diff/20001/sdk/lib/_internal/js_ru...
File sdk/lib/_internal/js_runtime/lib/js_rti.dart (right):

https://codereview.chromium.org/1345413002/diff/20001/sdk/lib/_internal/js_ru...
sdk/lib/_internal/js_runtime/lib/js_rti.dart:97: // creates an object that
already has the correct info.
do you mean that we would never invoke setRuntimeTypeInfo on the result of a
factory method, but that the call would happen when the actual allocation is
made? Or are we calling setRuntimeTypeInfo twice in that case?

[sra1, 2015-09-17 16:44:45]

Committed patchset #2 (id:40001) manually as
a2239c2deeeb8e8fd1d6fe5d1fde6dc275d576b3 (presubmit successful).