Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error in generated javascript (from InChI library) #13

Closed
baoilleach opened this issue May 7, 2011 · 3 comments
Closed

Error in generated javascript (from InChI library) #13

baoilleach opened this issue May 7, 2011 · 3 comments
Assignees

Comments

@baoilleach
Copy link

The InChI library is a somewhat complex C library used in chemistry.

The generated javascript is available at http:.//www.redbrick.dcu.ie/~noel/tmp/tmp.js.gz

"v8/d8 tmp.js" gives a Javascript error as follows:

$ ../v8-repo/d8 tmp.js
tmp.js:417618: SyntaxError: Unexpected token {
IHEAP[cnMe+6] = {$2{i16_2__i8_16__i8_0__i8_0__i8_undef}__%2_{i16_4__i8

It put a carat "^" underneath the first "{".

I'd really appreciate some help with this. The InChI library is available from http://www.iupac.org/inchi/download/version1.03/INCHI-1-API.zip. The makefile in INCHI-1-API/INCHI_API/gcc_so_makefile should be run as follows:

LINKER=pathto/emmaken.py SHARED_LINK=pathto/emmaken.py C_COMPILER=pathto/emmaken.py make

after commenting out the section "=== InChI Main Link rule ================" (whose compilation causes an llvm error).

@baoilleach
Copy link
Author

The corresponding bytecode is at http://www.redbrick.dcu.ie/~noel/tmp/libinchi.so.1.03.00.ll.gz
The link for the javascript is http://www.redbrick.dcu.ie/~noel/tmp/tmp.js.gz

@ghost ghost assigned kripken May 10, 2011
@kripken
Copy link
Member

kripken commented May 10, 2011

Thanks for the detailed bug report!

I fixed the compilation failure in commit 48caeb1. The generated script will now run without error (but it doesn't print anything, I didn't look into that).

@kripken kripken closed this as completed May 10, 2011
@baoilleach
Copy link
Author

That's great, thanks!

tlively pushed a commit to tlively/emscripten that referenced this issue Mar 23, 2022
…lder

Rename special folder to `persistent/` and add support for changing the folder name
steveisok pushed a commit to steveisok/emscripten that referenced this issue Sep 18, 2023
…728.2 (emscripten-core#13)

[dotnet/main] Update dependencies from dotnet/arcade
eagleoflqj pushed a commit to eagleoflqj/emscripten that referenced this issue Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants