Allow runtime switching between trans backends by bjorn3 · Pull Request #45684 · rust-lang/rust

bjorn3 · 2017-11-01T13:31:24Z

The driver callback after_llvm has been removed as it doesnt work with multiple backends.

eddyb · 2017-11-01T13:43:25Z

cc @alexcrichton @rust-lang/compiler I'm not sure a flag is the right thing to use here.
Long-term, do we expect/want to have -C backend=cranelift/llvm/etc. or different drivers?

bjorn3 · 2017-11-01T14:00:37Z

Most of the driver is the same for each backend. We could pass different compiler callbacks to rustc_driver for each backend.

alexcrichton · 2017-11-01T15:21:13Z

This seems to add -Z trans but never uses it? How can this be implemented unless we use trait objects?

bjorn3 · 2017-11-01T15:41:49Z

-Z trans is used at https://github.com/bjorn3/rust/blob/67f54e64464ae444de565cc48dc83c7abe6fd55e/src/librustc_driver/lib.rs#L244

eddyb · 2017-11-01T21:13:30Z

@bjorn3 We use "custom drivers" to refer to different binaries using rustc_driver, not replacements for rustc_driver.

bjorn3 · 2017-11-02T18:45:49Z

Now with less object unsafe associated types 🎊

carols10cents · 2017-11-06T14:14:13Z

What's the status of this PR @bjorn3 @alexcrichton @eddyb? I'm having a tough time telling :)

eddyb · 2017-11-06T14:16:20Z

I'm not sure what we should do here. Nominating for discussion in the next compiler meeting.

bjorn3 · 2017-11-07T18:05:15Z

Now with less object unsafe associated types 🎊

Edit: now completely object safe

eddyb · 2017-11-09T22:05:44Z

@bjorn3 Out of curiosity, do you have some specific motivation/need behind this PR, or is it intended as a general refactoring towards supporting multiple backends?

bjorn3 · 2017-11-10T07:11:29Z

Mainly refactoring, but when this is merged I want to try to add basic cranelift support.

eddyb · 2017-11-10T08:19:57Z

I'd suggest more coordination between you, @sunfishcode and me, and the compiler team in general. Just to avoid stepping on eachother's toes, as we are all interested in this goal :).

eddyb · 2017-11-12T05:22:55Z

r? @nikomatsakis

bjorn3 · 2017-11-12T09:35:32Z

Removed as it requires the now removed after_llvm callback.

nikomatsakis

So, eddyb kicked this to me, but I actually don't know that I'm the right person to do a detailed review here. I mean I could, but I think that @eddyb and perhaps @michaelwoerister have stronger opinions about how this should be internally architected. @bjorn3 can you and @eddyb sync up at some point and talk it over?

nikomatsakis · 2017-11-13T17:17:42Z

This seems like an important thing to get right before we land, no? Although I guess it's only a warning, not an error, but I wouldn't want to silently be accepting bad things.

bors · 2017-11-15T10:34:45Z

☔ The latest upstream changes (presumably #45944) made this pull request unmergeable. Please resolve the merge conflicts.

eddyb · 2017-11-15T15:02:52Z

You shouldn't need to do anything here, this is redundant.

You mean i should remove this line?

nikomatsakis · 2017-11-15T16:58:00Z

r? @eddyb -- I'm kicking this back to @eddyb =)

bors · 2018-01-19T22:38:08Z

💔 Test failed - status-travis

arielb1 · 2018-01-19T22:54:42Z

[01:09:03] error: internal compiler error: librustc/ty/maps/mod.rs:81: tcx.maps.target_features_enabled(DefId(0/0:1 ~ some_crate[317d]::{{?}}[0])) unsupported by its crate
[01:09:03] 
[01:09:03] note: the compiler unexpectedly panicked. this is a bug.
[01:09:03] 
[01:09:03] note: we would appreciate a bug report: https://github.com/rust-lang/rust/blob/master/CONTRIBUTING.md#bug-reports
[01:09:03] 
[01:09:03] note: rustc 1.25.0-dev running on x86_64-unknown-linux-gnu
[01:09:03] 
[01:09:03] thread 'rustc' panicked at 'Box<Any>', librustc_errors/lib.rs:509:9
[01:09:03] note: Run with `RUST_BACKTRACE=1` for a backtrace.
[01:09:03] 
[01:09:03] make[1]: *** [all] Error 101
[01:09:03] 
[01:09:03] ------------------------------------------
[01:09:03] 
[01:09:03] thread '[run-make] run-make/hotplug_codegen_backend' panicked at 'explicit panic', tools/compiletest/src/runtest.rs:2884:9
[01:09:03] note: Run with `RUST_BACKTRACE=1` for a backtrace.
[01:09:03] 
[01:09:03] 
[01:09:03] failures:
[01:09:03]     [run-make] run-make/hotplug_codegen_backend
[01:09:03]

bjorn3 · 2018-01-20T10:56:16Z

Fixed ICE

eddyb · 2018-01-20T10:57:16Z

@bors r+

bors · 2018-01-20T10:57:17Z

📌 Commit a4854e8 has been approved by eddyb

bors · 2018-01-20T11:02:02Z

⌛ Testing commit a4854e8 with merge 9aa40b0...

@eddyb

Allow runtime switching between trans backends The driver callback after_llvm has been removed as it doesnt work with multiple backends. r? @eddyb

bors · 2018-01-20T12:28:10Z

💔 Test failed - status-travis

eddyb · 2018-01-20T16:30:57Z

@bors r+

bors · 2018-01-20T16:30:57Z

📌 Commit a30232f has been approved by eddyb

bors · 2018-01-20T17:52:01Z

⌛ Testing commit a30232f with merge 3153d4a27cf582356da0bb5a2190c7bebd150372...

bors · 2018-01-20T20:52:28Z

💔 Test failed - status-travis

shepmaster · 2018-01-21T01:30:59Z

The job exceeded the maximum time limit for jobs, and has been terminated.

@bors retry

bors · 2018-01-21T09:17:32Z

⌛ Testing commit a30232f with merge 9368a1e...

@eddyb

Allow runtime switching between trans backends The driver callback after_llvm has been removed as it doesnt work with multiple backends. r? @eddyb

bors · 2018-01-21T12:05:44Z

☀️ Test successful - status-appveyor, status-travis
Approved by: eddyb
Pushing 9368a1e to master...

@nrc

Tested on commit rust-lang/rust@9368a1e. 💔 rls on windows: test-pass → build-fail (cc @nrc). 💔 rls on linux: test-pass → build-fail (cc @nrc).

rust-highfive assigned eddyb Nov 1, 2017

bjorn3 mentioned this pull request Nov 1, 2017

[driver breaking] Rename rustc_trans to rustc_codegen_llvm and allow runtime switching between trans backends #45677

Closed

kennytm added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Nov 1, 2017

bjorn3 commented Nov 2, 2017

View reviewed changes

Comment thread src/librustc_trans_utils/trans_crate.rs Outdated

bjorn3 Nov 2, 2017

Copy link
Copy Markdown

Member Author

Choose a reason for hiding this comment

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

Now with less object unsafe associated types 🎊

eddyb added the I-nominated label Nov 6, 2017

bjorn3 commented Nov 7, 2017

View reviewed changes

bjorn3 force-pushed the runtime_choose_trans2 branch from eae208e to fd38507 Compare November 11, 2017 16:51

rust-highfive assigned nikomatsakis and unassigned eddyb Nov 12, 2017

bjorn3 commented Nov 12, 2017

View reviewed changes

nikomatsakis reviewed Nov 14, 2017

View reviewed changes

bjorn3 force-pushed the runtime_choose_trans2 branch from 831a965 to b3a986a Compare November 15, 2017 14:30

eddyb reviewed Nov 15, 2017

View reviewed changes

rust-highfive assigned eddyb and unassigned nikomatsakis Nov 15, 2017

nikomatsakis removed the I-nominated label Nov 16, 2017

Fix ICE

a4854e8

bors added a commit that referenced this pull request Jan 20, 2018

Auto merge of #45684 - bjorn3:runtime_choose_trans2, r=eddyb

9aa40b0

Allow runtime switching between trans backends The driver callback after_llvm has been removed as it doesnt work with multiple backends. r? @eddyb

Remove the 'extern "C"' in the right place

a30232f

bjorn3 force-pushed the runtime_choose_trans2 branch from fb976fb to a30232f Compare January 20, 2018 16:27

bors added a commit that referenced this pull request Jan 21, 2018

Auto merge of #45684 - bjorn3:runtime_choose_trans2, r=eddyb

9368a1e

Allow runtime switching between trans backends The driver callback after_llvm has been removed as it doesnt work with multiple backends. r? @eddyb

bors merged commit a30232f into rust-lang:master Jan 21, 2018

This was referenced Jan 21, 2018

rustc: Lower link args to @-files on Windows more #47507

Merged

rustc: Add -C lto=val option #47521

Merged

Implement RFC 1946 - intra-rustdoc links #47046

Merged

WIP: Parallelize passes using rayon #46564

Closed

topecongiro mentioned this pull request Jan 22, 2018

Rustup to rustc 1.25.0-nightly (97520ccb1 2018-01-21) rust-dev-tools/rls-rustc#5

Merged

alexcrichton mentioned this pull request Jan 23, 2018

rustc: Load the rustc_trans crate at runtime #47671

Merged

XAMPPRocky mentioned this pull request Oct 14, 2020

Tracking Issue for -Z codegen-backend #77933

Open

Uh oh!

Conversation

bjorn3 commented Nov 1, 2017

Uh oh!

eddyb commented Nov 1, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bjorn3 commented Nov 1, 2017

Uh oh!

alexcrichton commented Nov 1, 2017

Uh oh!

bjorn3 commented Nov 1, 2017

Uh oh!

eddyb commented Nov 1, 2017

Uh oh!

bjorn3 Nov 2, 2017

Choose a reason for hiding this comment

Uh oh!

carols10cents commented Nov 6, 2017

Uh oh!

eddyb commented Nov 6, 2017

Uh oh!

bjorn3 Nov 7, 2017

Choose a reason for hiding this comment

Uh oh!

eddyb commented Nov 9, 2017

Uh oh!

bjorn3 commented Nov 10, 2017 • edited by eddyb Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

eddyb commented Nov 10, 2017

Uh oh!

eddyb commented Nov 12, 2017

Uh oh!

bjorn3 Nov 12, 2017

Choose a reason for hiding this comment

Uh oh!

nikomatsakis left a comment

Choose a reason for hiding this comment

Uh oh!

nikomatsakis Nov 13, 2017

Choose a reason for hiding this comment

Uh oh!

bors commented Nov 15, 2017

Uh oh!

eddyb Nov 15, 2017

Choose a reason for hiding this comment

Uh oh!

bjorn3 Nov 15, 2017

Choose a reason for hiding this comment

Uh oh!

eddyb Nov 15, 2017

Choose a reason for hiding this comment

Uh oh!

nikomatsakis commented Nov 15, 2017

Uh oh!

bors commented Jan 19, 2018

Uh oh!

arielb1 commented Jan 19, 2018

Uh oh!

bjorn3 commented Jan 20, 2018

Uh oh!

eddyb commented Jan 20, 2018

Uh oh!

bors commented Jan 20, 2018

Uh oh!

bors commented Jan 20, 2018

Uh oh!

bors commented Jan 20, 2018

Uh oh!

eddyb commented Jan 20, 2018

Uh oh!

bors commented Jan 20, 2018

Uh oh!

bors commented Jan 20, 2018

Uh oh!

bors commented Jan 20, 2018

Uh oh!

shepmaster commented Jan 21, 2018

Uh oh!

eddyb commented Nov 1, 2017 •

edited

Loading

bjorn3 commented Nov 10, 2017 •

edited by eddyb

Loading