Skip to content

Conversation

alexcrichton
Copy link
Member

This commit updates the compiler-builtins submodule, primarily pulling in
rust-lang/compiler-builtins#248 which should resolve a number of
undefined symbols on the wasm target by default (instead of leaving them in the
final binary).

This commit updates the compiler-builtins submodule, primarily pulling in
rust-lang/compiler-builtins#248 which should resolve a number of
undefined symbols on the wasm target by default (instead of leaving them in the
final binary).
@rust-highfive
Copy link
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link
Contributor

warning Warning warning

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 18, 2018
@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 18, 2018

📌 Commit 8fb918b has been approved by nikomatsakis

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 18, 2018
@kennytm
Copy link
Member

kennytm commented Jul 18, 2018

How about #52431, should that be closed?

@alexcrichton
Copy link
Member Author

Aha excellent point, I've moved this over to #52431 by updating the submodule a bit further there

@alexcrichton alexcrichton deleted the update-compiler-builtins branch July 18, 2018 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants