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

Improve bytecode compilation #66

Merged
merged 3 commits into from
Nov 9, 2017
Merged

Improve bytecode compilation #66

merged 3 commits into from
Nov 9, 2017

Conversation

tomdale
Copy link
Contributor

@tomdale tomdale commented Nov 9, 2017

This PR is a grab-bag of improvements to the toolchain for compiling applications to Glimmer bytecode.

In particular, this PR:

  1. Updates Glimmer VM to 0.30.1.
  2. Updates the Module Unification compiler delegate (@glimmer/compiler-delegates) to use the new ModuleLocator/TemplateLocator system for addressing buildtime metadata, and being more rigorous about handling runtime template metadata.
  3. Supports a first-class notion of "builtins" in the compiler, allowing Glimmer.js and other extensions to prepopulate the compiler with well-known helpers and templates that can be reified at runtime.
  4. Supports encoding type information in the external module table, which allows us to shrink the size of the serialized table considerably.

},
"devDependencies": {
"@glimmer/application-test-helpers": "^0.9.0-alpha.4",
"@glimmer/component": "^0.9.0-alpha.4",
"@glimmer/wire-format": "^0.29.10",
"@glimmer/test-helpers": "^0.30.1",
Copy link
Member

Choose a reason for hiding this comment

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

Remove this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@@ -210,6 +210,7 @@ if (DEBUG) {
.template('Main', '<h1>Hello, {{name}}!</h1>')
.boot();
} catch (err) {
console.log(err.message);
Copy link
Member

Choose a reason for hiding this comment

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

Remove

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@chadhietala chadhietala self-requested a review November 9, 2017 15:43
@chadhietala chadhietala merged commit 1e1d312 into master Nov 9, 2017
@chadhietala chadhietala deleted the bytecode-compatibility branch November 9, 2017 16:15
Turbo87 pushed a commit to Turbo87/glimmer.js that referenced this pull request Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants