Minor fixes for background compilation

Minor fixes for background compilation

BUG=
R=asiva@google.com

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

[srdjan, 2016-02-17 22:49:04]

srdjan@google.com changed reviewers:
+ asiva@google.com

[siva, 2016-02-17 23:12:48]

LGTM with one comment.

https://codereview.chromium.org/1708013002/diff/1/runtime/vm/compiler.cc
File runtime/vm/compiler.cc (right):

https://codereview.chromium.org/1708013002/diff/1/runtime/vm/compiler.cc#newc...
runtime/vm/compiler.cc:609: FLAG_max_deoptimization_counter_threshold);
maybe put parenthesis around (function.dep....() < FLAG_max_de...));

https://codereview.chromium.org/1708013002/diff/1/runtime/vm/stub_code.cc
File runtime/vm/stub_code.cc (right):

https://codereview.chromium.org/1708013002/diff/1/runtime/vm/stub_code.cc#new...
runtime/vm/stub_code.cc:128: cls.set_allocation_stub(stub);
As discussed offline you might need a check here too.

[srdjan, 2016-02-17 23:32:18]

Description was changed from

==========
Minor fixes for background compilation

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

to

==========
Minor fixes for background compilation

BUG=
R=asiva@google.com

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

[srdjan, 2016-02-17 23:32:19]

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

[srdjan, 2016-02-17 23:32:29]

https://codereview.chromium.org/1708013002/diff/1/runtime/vm/compiler.cc
File runtime/vm/compiler.cc (right):

https://codereview.chromium.org/1708013002/diff/1/runtime/vm/compiler.cc#newc...
runtime/vm/compiler.cc:609: FLAG_max_deoptimization_counter_threshold);
On 2016/02/17 23:12:48, siva wrote:
> maybe put parenthesis around (function.dep....() < FLAG_max_de...));

Done.

https://codereview.chromium.org/1708013002/diff/1/runtime/vm/stub_code.cc
File runtime/vm/stub_code.cc (right):

https://codereview.chromium.org/1708013002/diff/1/runtime/vm/stub_code.cc#new...
runtime/vm/stub_code.cc:128: cls.set_allocation_stub(stub);
On 2016/02/17 23:12:48, siva wrote:
> As discussed offline you might need a check here too.

Done.