A collection of patches extracted from the Tracemonkey branch and rebased to apply against tamarin redux. The patches will be posted in order of application and rely on the landing of early patches outlined in 515539, 515548 and 515822 respectively. Each patch represents a check-in and associated patch from the Tracemonkey branch. Bug numbers will be specified for each of the patches.

Comment on attachment 401285 [details] [diff] [review] bug 514537 Please don't re-add the NJ_LOG2_PAGE_SIZE = 12 constant in NativeX64.h

Comment on attachment 401288 [details] [diff] [review] bug 514503 - sanity filter This is based on the ValidateFilter in CodegenLIR.cpp, but is missing a bunch of other instruction categories. lets make sure we remove the duplicate code from ValidateFilter ASAP.

Comment on attachment 401301 [details] [diff] [review] bug 514532 - lir 64b helpers looks like we have some redundant helpers in CodegenLIR.cpp that can be removed due to this patch (i'm thinking of u2p() and i2p())

Comment on attachment 401306 [details] [diff] [review] bug 513787 - epilogue and jumptable curious how this ends up looking on tamarin where we have >1 LIR_ret per method (it occurs each place an OP_returnvalue occurs in ABC)

re comment #18 : done re comment #19 : created bug 517561 re comment #20 : created bug 517560 re comment #21 : doesn't seem to move the needle for us (at least on the tests I've run) but TM reports a modest gain

pushed en masse starting from : http://hg.mozilla.org/tamarin-redux/rev/504bcef895e8 ending at: http://hg.mozilla.org/tamarin-redux/rev/de7a597eea3a

Resolved fixed engineering / work item that has been pushed. Setting status to verified.