search

cesanta / v7

Embedded JavaScript engine for C/C++
Other
1.42k stars 177 forks source link

Cross compile V7 and failed to run on mips-32bit #547

Open ghost opened 8 years ago

ghost commented 8 years ago

./v7: v7.c: 9223: calc_llen: Assertion `n <= (int) sizeof(len)' failed. Aborted

goniz commented 8 years ago

I'm experiencing this problem as well. Can a fix be implemented easily?

mkmik commented 8 years ago

big endian?

goniz commented 8 years ago

Yes, big endian.

On Thu, Jun 9, 2016, 00:52 Marko Mikulicic notifications@github.com wrote:

big endian?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cesanta/v7/issues/547#issuecomment-224740930, or mute the thread https://github.com/notifications/unsubscribe/AA7no7Zn2bXs3WFLUI21R5G_Bn3qvVzGks5qJzmpgaJpZM4HvLja .

goniz commented 8 years ago

Is this a 32bit vs 64bit issue? or a big endian vs little endian issue?

dimonomid commented 8 years ago

v7 runs perfectly fine on both 64bit and 32bit. This is a big-endian issue.

goniz commented 8 years ago

running ./v7_mips -e 'print(1);' after commit 91118131613f78037df1c71126e388d1a01ed443 give the following output:

"cannot find exception TypeError Exception creation throws an exception TypeError: cannot read property 'prototype' of undefined Assertion failed: v7_is_object(v) (v7.c: get_object_struct: 17840)"

dimonomid commented 8 years ago

Yes, we're aware of the issues on big-endian platforms; honestly we never addressed them yet. But yes, this thing is important, and we plan to fix it in July 2016 (can't do that earlier due to our local deadlines). We'll report here once we have any progress on that.

goniz commented 8 years ago

Thank you for the quick response, July sounds great! I will be following your progress :+1:

goniz commented 8 years ago

Looking forward for the fix :)