mozilla / pluotsorbet

[ARCHIVED] PluotSorbet is a J2ME-compatible virtual machine written in JavaScript.
GNU General Public License v2.0
237 stars 46 forks source link

Add option to disable null checks in the compiler. #1853

Closed mbebenita closed 6 years ago

mbebenita commented 9 years ago

This makes a huge difference on some micro-benchmarks. We should investigate why. I suspect that it has to do with some optimizations being prevented by calls to TN().

BubbleSort with 1000 iterations goes from 2969ms to 705ms. HotSpot is about 300ms.

mykmelez commented 9 years ago

The test failure is:

1.2:1.5 | Running gnu.testlet.java.io.DataInputStreamTest 1.2:1.5 | gnu.testlet.java.io.DataInputStreamTest-16: fail 1.2:1.5 | gnu.testlet.java.io.DataInputStreamTest: 43 pass, 1 fail, 0 known fail, 0 unknown pass 1.2:1.5 | gnu.testlet.java.io.DataInputStreamTest: test expected 44 passes, got 43 1.2:1.5 | gnu.testlet.java.io.DataInputStreamTest: test expected 0 failures, got 1 1.2:1.5 | gnu.testlet.java.io.DataInputStreamTest: class fail

mykmelez commented 6 years ago

I'm closing this pull request (and all others) as this project is no longer active.