Closed Bug 679794 Opened 10 years ago Closed 10 years ago
Monkey: Assertion failure: a .is General Reg(), at shared/Code Generator-shared-inl .h:51 (inside Code Generator X64::visit Double)
Asserts on attached test case on IonMonkey, 64-bit with greedy register allocator.
previous patch cause the following failure lately. The patch attached fix this new assertions and the test case is now running as expected. Assertion failure: from->isDouble() == to->isDouble(), at /mnt/data/home/nicolas/mozilla/ionmonkey/js/src/ion/shared/CodeGenerator-x86-shared.cpp:292
Attachment #553840 - Flags: review?
Attachment #553840 - Flags: review? → review?(dvander)
Comment on attachment 553836 [details] [diff] [review] Fix reported bug, temporary use a normal register to load immediate to xmm register. Nice catch.
Attachment #553836 - Flags: review?(dvander) → review+
The second patch appears to be empty - No review needed for test cases, just fold them into a patch. You can throw them in js/src/jit-tests/tests/ion
This patch contains the test case and the 2 previous patches (including the content of the missing one). This fix the bug for the greedy allocator on x86_64, It does not fix the assertion  raised when used with the linear scan register allocator.  Assertion failure: a.isFloatReg(), at ../ion/shared/CodeGenerator-shared-inl.h:70
Attachment #554023 - Flags: review?(dvander) → review+
(Turns out the temp-double part of the patch had been obsoleted by bug 677339, but the rest applied.) http://hg.mozilla.org/projects/ionmonkey/rev/6050e64e2903
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Duplicate of this bug: 678652
You need to log in before you can comment on or make changes to this bug.