843023 - IonMonkey: inline Math.pow with integer result

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Hannes Verschore [:h4writer]]

Currently Math.pow gets inlined when the result type is double. We should do the same for Int32 result type, because now we do a call for these.

Comment 1

[Hannes Verschore [:h4writer]]

Attachment: Inline Math.pow when result is expected to be intger

Quick patch to do this. I noticed there are some places in benchmarks that do this. So should be minor/really small win.

I see a 10x improvement on:

eval("");
function pow(a, b) { return Math.pow(a, b);}
for (var i=0; i<100000000; i++)
  pow(1, 5);

Comment 2

[Hannes Verschore [:h4writer]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/04ce59868888

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/04ce59868888