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

Add results for GraalVM JavaScript #1323

Merged
merged 3 commits into from
Jul 10, 2018
Merged

Add results for GraalVM JavaScript #1323

merged 3 commits into from
Jul 10, 2018

Conversation

wirthi
Copy link
Contributor

@wirthi wirthi commented Jul 6, 2018

Hi,

GraalVM (https://www.graalvm.org) is coming with JavaScript (and Node.js) support. This PR adds results for this engine.

GraalVM JavaScript is based on the Graal Java JIT compiler (https://github.com/oracle/graal). The source code is available at https://github.com/graalvm/graaljs

Supporting Node.js, GraalVM can reuse the node.js file to generate the results. The commandline to reproduce is /path/to/graalvm-ce-1.0.0-rc3/bin/node --jvm --shared-array-buffer --intl-402 node.js. The results are based on GraalVM CE 1.0.0 RC3 (https://github.com/oracle/graal/releases/tag/vm-1.0.0-rc3).

-- Christian

@ljharb
Copy link
Member

ljharb commented Jul 6, 2018

(the link has an SSL error btw)

@ljharb
Copy link
Member

ljharb commented Jul 6, 2018

LGTM - could you add npm run-script(s) so that the commands to generate the results (and install the needed deps) are discoverable in the repo?

@wirthi
Copy link
Contributor Author

wirthi commented Jul 6, 2018

Thanks, fixed the link.

What run scripts do you expect exactly? GraalVM is compatible with Node.js, so all the pieces are in the repo already - you can use the node.js file to generate the results with GraalVM as described above.

@ljharb
Copy link
Member

ljharb commented Jul 6, 2018

like, npm run graal or something :-) just like https://github.com/kangax/compat-table/blob/gh-pages/package.json#L43

@wirthi
Copy link
Contributor Author

wirthi commented Jul 6, 2018

ah, ok, gotcha, will add.

@wirthi
Copy link
Contributor Author

wirthi commented Jul 6, 2018

I've added a script so you can do npm run graalvm. Note however that this assumes that GraalVM is on your path before Node.js. If this is not the case, you need to explicitly call GraalVM's node command as described above.

@@ -42,7 +42,8 @@
"pretest": "npm run lint",
"build:compilers": "node build.js compilers",
"git:clean": "if type git > /dev/null 2>&1; then git stash save --keep-index; fi",
"test:no_change": "npm run --silent git:clean && if npm run --silent build | grep 'Write to file'; then echo 'Please run `npm run build` and commit the changes.' && exit 1; fi"
"test:no_change": "npm run --silent git:clean && if npm run --silent build | grep 'Write to file'; then echo 'Please run `npm run build` and commit the changes.' && exit 1; fi",
"graalvm": "node --jvm --shared-array-buffer --intl-402 node.js"

This comment was marked as resolved.

@pmlopes
Copy link
Contributor

pmlopes commented Jul 8, 2018

I think the results should state that this was run on the nodejs mode, not embedded mode.

@wirthi
Copy link
Contributor Author

wirthi commented Jul 10, 2018

I have added such a comment.

Copy link
Member

@chicoxyzzy chicoxyzzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, looks good

@chicoxyzzy chicoxyzzy merged commit 956d186 into compat-table:gh-pages Jul 10, 2018
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

Successfully merging this pull request may close these issues.

None yet

4 participants