-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Output of independent artefacts might differ depending on unrelated options #22183
Labels
A-CLI
Area: Command-line interface (CLI) to the compiler
C-bug
Category: This is a bug.
I-needs-decision
Issue: In need of a decision.
T-compiler
Relevant to the compiler team, which will review and decide on the PR/issue.
Comments
The check whether the entry point is necessary or not should occur elsewhere. |
nagisa
added a commit
to nagisa/rust
that referenced
this issue
Feb 11, 2015
This resulted in some very unexpected fallout such as empty llvm-ir, llvm-bc and asm in some cases. See rust-lang#22183 for related info.
Merged
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Feb 11, 2015
This resulted in some very unexpected fallout such as empty llvm-ir, llvm-bc and asm in some cases. See rust-lang#22183 for related info. r? @alexcrichton
Triage: not aware of any changes here? |
I think that not much has changed here. The individual outputs are different, but the overall diff is the same: entry points are included in --- no-bin 2017-04-30 18:10:53.442645721 -0600
+++ bin 2017-04-30 18:10:48.398648634 -0600
@@ -1,5 +1,8 @@
test.0.o:
+0000000000000000 T main
+ U std::rt::lang_start::h8126425a59d3b4a0
+0000000000000000 t test::main::hf4d700377696cbb0
std-35ad9950c7e5074b.0.o:
U abort
|
I'm not sure why this is marked as rustbuild, it seems to affect all binaries. Going to move this to T-compiler. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-CLI
Area: Command-line interface (CLI) to the compiler
C-bug
Category: This is a bug.
I-needs-decision
Issue: In need of a decision.
T-compiler
Relevant to the compiler team, which will review and decide on the PR/issue.
Existence or lack of
--crate-type=bin
may influence output of othercrate-type
outputs. In generallibfoo.a
output by--crate-type=bin,staticlib
and--crate-type=staticlib
might be completely different.For example using this
foo.rs
, this is the list of a few first symbols when built with--crate-type=bin,staticlib
and this, whenbin
crate-type is absent.This is a quirk in rust’s entry point semantics and how current
middle/entry.rs
is implemented.The text was updated successfully, but these errors were encountered: