Skip to content

-o no longer works as expected for multiple output types #40993

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
nagisa opened this issue Apr 1, 2017 · 4 comments
Closed

-o no longer works as expected for multiple output types #40993

nagisa opened this issue Apr 1, 2017 · 4 comments
Assignees
@nagisa
Labels
P-high High priority regression-from-stable-to-stable Performance or correctness regression from one stable version to another. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@nagisa
Copy link
Member

nagisa commented Apr 1, 2017 

$ rustc test.rs --emit=asm,llvm-ir -O --crate-type=rlib -o testold

emits single file testold with LLVM-IR in it. Previously it used to emit testold.s and testold.ll.

This is a stable→stable regression. 1.14.0 works as expected, 1.16.0 does not. Did not test 1.15.0.
@alexcrichton alexcrichton added the regression-from-stable-to-stable Performance or correctness regression from one stable version to another. label Apr 1, 2017
@arielb1
Copy link
Contributor

arielb1 commented Apr 2, 2017 •
edited
Loading

"Intentional" change - #38840. Not marked relnotes. cc @nrc

@alexcrichton
Copy link
Member

@arielb1 did you perhaps mean to link a different PR?

@brson brson added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Apr 4, 2017
@brson
Copy link
Contributor

brson commented Apr 4, 2017

#38840

@brson brson added the P-high High priority label Apr 4, 2017
@brson
Copy link
Contributor

brson commented Apr 4, 2017

I took the liberty of P-high, but somebody needs to be assigned cc @rust-lang/compiler @rust-lang/tools

@brson brson assigned nagisa and unassigned nagisa Apr 4, 2017
nagisa added a commit to nagisa/rust that referenced this issue Apr 5, 2017
@nagisa
Properly adjust filenames when multiple emissions
201b1a9 
Fixes rust-lang#40993
@nagisa nagisa mentioned this issue Apr 5, 2017
Merged
frewsxcv added a commit to frewsxcv/rust that referenced this issue Apr 5, 2017
@frewsxcv
Rollup merge of rust-lang#41085 - nagisa:fix-output-properg, r=alexcr…
02ff7d1 
…ichton

Properly adjust filenames when multiple emissions

Fixes rust-lang#40993

Should backport just fine to beta but not sure if we want to do this since this is quite old stable regression.
frewsxcv added a commit to frewsxcv/rust that referenced this issue Apr 5, 2017
@frewsxcv
Rollup merge of rust-lang#41085 - nagisa:fix-output-properg, r=alexcr…
97a1f4b 
…ichton

Properly adjust filenames when multiple emissions

Fixes rust-lang#40993

Should backport just fine to beta but not sure if we want to do this since this is quite old stable regression.
alexcrichton pushed a commit to alexcrichton/rust that referenced this issue Apr 20, 2017
@nagisa @alexcrichton
Properly adjust filenames when multiple emissions
1364ffc 
Fixes rust-lang#40993
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@nagisa nagisa

Labels
P-high High priority regression-from-stable-to-stable Performance or correctness regression from one stable version to another. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@alexcrichton @brson @nagisa @arielb1