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

Pass debugging arguments to emcc #43344

Merged
merged 1 commit into from
Jul 22, 2017
Merged

Pass debugging arguments to emcc #43344

merged 1 commit into from
Jul 22, 2017

Conversation

tlively
Copy link
Contributor

@tlively tlively commented Jul 19, 2017

Tells emcc to enable assertions and debugging information for
wasm32-experimental-emscripten. This makes the codegen issues caused by
LLVM bug 33824 manifest more frequently at runtime and improves the wasm
debugging experience.

Tells emcc to enable assertions and debugging information for
wasm32-experimental-emscripten. This makes the codegen issues caused by
LLVM bug 33824 manifest more frequently at runtime and improves the wasm
debugging experience.
@rust-highfive
Copy link
Collaborator

r? @eddyb

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

@tlively
Copy link
Contributor Author

tlively commented Jul 19, 2017

r? @alexcrichton

@rust-highfive rust-highfive assigned alexcrichton and unassigned eddyb Jul 19, 2017
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Jul 19, 2017

📌 Commit 6a3e322 has been approved by alexcrichton

@aidanhs aidanhs added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 20, 2017
@bors
Copy link
Contributor

bors commented Jul 21, 2017

⌛ Testing commit 6a3e322 with merge 8856e751e67b6301c2ad5bb2f7f8f6af80baa54a...

@bors
Copy link
Contributor

bors commented Jul 21, 2017

💔 Test failed - status-travis

@aidanhs
Copy link
Member

aidanhs commented Jul 21, 2017

[ERROR]  >>  Build failed in step 'Retrieving needed toolchain components' tarballs'
[00:06:10] 
[04:41] / 
[ERROR]  >>        called in step '(top-level)'
[00:06:10] 
[04:41] / 
[ERROR]  >>
[00:06:10] 
[04:41] / 
[ERROR]  >>  Error happened in: do_isl_get[scripts/build/companion_libs/121-isl.sh@741]
[00:06:10] 
[04:41] / 
[ERROR]  >>        called from: do_companion_libs_get[scripts/build/companion_libs.sh@15]
[00:06:10] 
[04:41] / 
[ERROR]  >>        called from: main[scripts/crosstool-NG.sh@591]

Hmm, not sure about this one. Sounds like a failure while downloading something? So

@bors retry

@bors
Copy link
Contributor

bors commented Jul 21, 2017

⌛ Testing commit 6a3e322 with merge 969dd6ac0b7e66f7670005e3035ff5fff3146586...

@bors
Copy link
Contributor

bors commented Jul 21, 2017

💔 Test failed - status-appveyor

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Jul 22, 2017

⌛ Testing commit 6a3e322 with merge e49a948...

bors added a commit that referenced this pull request Jul 22, 2017
Pass debugging arguments to emcc

Tells emcc to enable assertions and debugging information for
wasm32-experimental-emscripten. This makes the codegen issues caused by
LLVM bug 33824 manifest more frequently at runtime and improves the wasm
debugging experience.
@bors
Copy link
Contributor

bors commented Jul 22, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing e49a948 to master...

@bors bors merged commit 6a3e322 into rust-lang:master Jul 22, 2017
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.

6 participants