Lars T Hansen [:lth] Reporter
	Description • 9 years ago

This should have been fixed by bug 1178793 but was not, very likely because the ARMv6 code uses call-outs that were not updated and we're not actually testing on ARMv6.

On a Raspberry Pi model 1:

pi@raspberrypi:~/moz/js/src/jit-test$ ./jit_test.py ~/js testAtomics.js
/home/pi/moz/js/src/jit-test/lib/asserts.js:85:15 Error: Assertion failed: expected exception RangeError, no exception thrown
Exit code: 3
FAIL - asm.js/testAtomics.js

	Lars T Hansen [:lth] Reporter
	Comment 1 • 9 years ago

Reproduce on simulator with ARMHWCAP=hardfp,vfp, which forces the ARMv6 path.

The problem is that Lowering-arm.cpp lowers to special "Callout" LIR nodes, and these need to have range checking logic as well.  The C++ methods do contain range checks but return 0.  The code generated for the callout can either explicitly check the range and throw, or it can perhaps check the result of the callout.

	Lars T Hansen [:lth] Reporter
	Comment 2 • 8 years ago

ARMv6 is Tier-3.

	Lars T Hansen [:lth] Reporter
	Comment 3 • 7 years ago

We ripped out the ARMv6 callout support some time ago.  We're no longer supporting that architecture, and MIPS generates in-line machine code for similar cases and does not use the callout support.