Support non-deoptimizing version of UnboxInt32 for integer definitions.

Support non-deoptimizing version of UnboxInt32 for integer definitions.

If range analysis infered that integer definition is within Int32 range then it is guaranteed to be either Smi or Mint.

BUG=http://dartbug.com/21763
R=fschneider@google.com

Committed: https://code.google.com/p/dart/source/detail?r=42065

[Vyacheslav Egorov (Google), 2014-12-02 12:59:46]

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

[Vyacheslav Egorov (Google), 2014-12-02 12:59:46]

PTAL

[Florian Schneider, 2014-12-02 13:12:04]

lgtm

https://codereview.chromium.org/770303002/diff/1/runtime/vm/intermediate_lang...
File runtime/vm/intermediate_language_arm.cc (right):

https://codereview.chromium.org/770303002/diff/1/runtime/vm/intermediate_lang...
runtime/vm/intermediate_language_arm.cc:3998: LoadInt32FromMint(compiler, value,
out, temp, out_of_range);
s/out_of_range/NULL/

since it is always NULL in this case?

https://codereview.chromium.org/770303002/diff/1/runtime/vm/intermediate_lang...
File runtime/vm/intermediate_language_mips.cc (right):

https://codereview.chromium.org/770303002/diff/1/runtime/vm/intermediate_lang...
runtime/vm/intermediate_language_mips.cc:3355: LoadInt32FromMint(compiler,
value, out, out_of_range);
Same comment as on ARM.

[Vyacheslav Egorov (Google), 2014-12-02 13:15:58]

https://codereview.chromium.org/770303002/diff/1/runtime/vm/intermediate_lang...
File runtime/vm/intermediate_language_arm.cc (right):

https://codereview.chromium.org/770303002/diff/1/runtime/vm/intermediate_lang...
runtime/vm/intermediate_language_arm.cc:3998: LoadInt32FromMint(compiler, value,
out, temp, out_of_range);
On 2014/12/02 13:12:03, Florian Schneider wrote:
> s/out_of_range/NULL/
> 
> since it is always NULL in this case?

Done.

https://codereview.chromium.org/770303002/diff/1/runtime/vm/intermediate_lang...
File runtime/vm/intermediate_language_mips.cc (right):

https://codereview.chromium.org/770303002/diff/1/runtime/vm/intermediate_lang...
runtime/vm/intermediate_language_mips.cc:3355: LoadInt32FromMint(compiler,
value, out, out_of_range);
On 2014/12/02 13:12:04, Florian Schneider wrote:
> Same comment as on ARM.

Done.

[Vyacheslav Egorov (Google), 2014-12-02 13:19:11]

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