Skip to content

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Nov 15, 2019

See rust-lang/llvm-project#27, which works around rust-lang/llvm-project#20 (comment) (where I stumbled over some UB in DWARF emission for WASM, resulting in non-deterministic output).

r? @alexcrichton cc @yurydelendik

@rust-highfive
Copy link
Contributor

⚠️ 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 Nov 15, 2019
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Nov 15, 2019

📌 Commit d137e8e has been approved by alexcrichton

@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 Nov 15, 2019
@bors
Copy link
Collaborator

bors commented Nov 18, 2019

⌛ Testing commit d137e8e with merge a0d40f8...

bors added a commit that referenced this pull request Nov 18, 2019
Update src/llvm-project to include rust-lang/llvm-project#27.

See rust-lang/llvm-project#27, which works around rust-lang/llvm-project#20 (comment) (where I stumbled over some UB in DWARF emission for WASM, resulting in non-deterministic output).

r? @alexcrichton cc @yurydelendik
@bors
Copy link
Collaborator

bors commented Nov 18, 2019

☀️ Test successful - checks-azure
Approved by: alexcrichton
Pushing a0d40f8 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Nov 18, 2019
@bors bors merged commit d137e8e into rust-lang:master Nov 18, 2019
@eddyb eddyb deleted the update-llvm-wasm-dwarf branch November 18, 2019 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.

4 participants