Skip to content

Pass debugging arguments to emcc#43344

Merged
bors merged 1 commit into
rust-lang:masterfrom
tlively:wasm-debug
Jul 22, 2017
Merged

Pass debugging arguments to emcc#43344
bors merged 1 commit into
rust-lang:masterfrom
tlively:wasm-debug

Conversation

@tlively

@tlively tlively commented Jul 19, 2017

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

r? @eddyb

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

@tlively

tlively commented Jul 19, 2017

Copy link
Copy Markdown
Contributor Author

r? @alexcrichton

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

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Jul 19, 2017

Copy link
Copy Markdown
Collaborator

📌 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

bors commented Jul 21, 2017

Copy link
Copy Markdown
Collaborator

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

@bors

bors commented Jul 21, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-travis

@aidanhs

aidanhs commented Jul 21, 2017

Copy link
Copy Markdown
Contributor
[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

bors commented Jul 21, 2017

Copy link
Copy Markdown
Collaborator

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

@bors

bors commented Jul 21, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-appveyor

@alexcrichton

Copy link
Copy Markdown
Member

@bors: retry

@bors

bors commented Jul 22, 2017

Copy link
Copy Markdown
Collaborator

⌛ 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

bors commented Jul 22, 2017

Copy link
Copy Markdown
Collaborator

☀️ 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