cttz_i8 is not defined [PATCH]

[oflebbe]

Hi,

had the same issue running my ASM_JS app with -O3.

(Like
https://groups.google.com/forum/#!topic/emscripten-discuss/HbTT01LESqs)

Testcase and proposed patch appended. I do not see the point adding the undef cttz_i8 with Imprecise_64 bit integers. The symbol seems only needed for llvm_cttz_ functions.

Sorry, testcase is a little big ... I am in the middle of optimizing and had no time to strip it down. See README for compile flags.

Patch https://www.dropbox.com/s/nqpsfc458jdnt6p/0001-Fix-cttz_i8-undefined-with-O3-and-ASM_JS.patch
Testcase https://www.dropbox.com/s/py9nl9dad3t6vnv/rochelobe.tgz

[kripken]

Is the relevant part I64_PRECISE_MATH=0? (of the flags that -O3 flips)

[oflebbe]

Not sure what you want me to do: I used emcc -O2 with -s PRECISE_I64_MATH=0 with unmodified emscripten and got an internal error:

....
Assertion failed: Must have precise i64 math for non-constant 64-bit shifts
....

Olaf

Am 08.07.2013 um 01:29 schrieb Alon Zakai notifications@github.com:

Is the relevant part I64_PRECISE_MATH=0? (of the flags that -O3 flips)

—
Reply to this email directly or view it on GitHub.

[kripken]

Thanks, that's what I meant. So that code must require precise i64 math. That flag, like all -O3 options, can break code, looks like in this case it can't be used.