Created attachment 769363 [details] micro-benchmark, unrolled
Attachment #769362 - Attachment is obsolete: true
Comment on attachment 769361 [details] [diff] [review] Micro-optimization of js::ToInteger Stumbled upon this. The attached micro-benchmark gets a speedup of ~9%. While I doubt that that makes a difference in any benchmarks we care about, simplifying code and getting rid of two branches can't be wrong. Right?
Attachment #769361 - Flags: review?(jwalden+bmo)
Attachment #769361 - Flags: review?(jwalden+bmo) → review+
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.