Skip to content

Commit d7e772d

Browse files
committed
Initial support for dynamically linked crates
1 parent 1f76d21 commit d7e772d

File tree

68 files changed

+1531
-117
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

68 files changed

+1531
-117
lines changed

compiler/rustc_ast_lowering/src/item.rs

+3-1
Original file line numberDiff line numberDiff line change
@@ -1310,7 +1310,9 @@ impl<'hir> LoweringContext<'_, 'hir> {
13101310
// create a fake body so that the entire rest of the compiler doesn't have to deal with
13111311
// this as a special case.
13121312
return self.lower_fn_body(decl, contract, |this| {
1313-
if attrs.iter().any(|a| a.name_or_empty() == sym::rustc_intrinsic) {
1313+
if attrs.iter().any(|a| a.name_or_empty() == sym::rustc_intrinsic)
1314+
|| this.tcx.is_sdylib_interface_build()
1315+
{
13141316
let span = this.lower_span(span);
13151317
let empty_block = hir::Block {
13161318
hir_id: this.next_id(),

compiler/rustc_ast_passes/src/ast_validation.rs

+6-2
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,8 @@ struct AstValidator<'a> {
8282
/// Used to ban explicit safety on foreign items when the extern block is not marked as unsafe.
8383
extern_mod_safety: Option<Safety>,
8484

85+
is_sdylib_interface: bool,
86+
8587
lint_buffer: &'a mut LintBuffer,
8688
}
8789

@@ -949,7 +951,7 @@ impl<'a> Visitor<'a> for AstValidator<'a> {
949951

950952
let is_intrinsic =
951953
item.attrs.iter().any(|a| a.name_or_empty() == sym::rustc_intrinsic);
952-
if body.is_none() && !is_intrinsic {
954+
if body.is_none() && !is_intrinsic && !self.is_sdylib_interface {
953955
self.dcx().emit_err(errors::FnWithoutBody {
954956
span: item.span,
955957
replace_span: self.ending_semi_or_hi(item.span),
@@ -1436,7 +1438,7 @@ impl<'a> Visitor<'a> for AstValidator<'a> {
14361438
});
14371439
}
14381440
AssocItemKind::Fn(box Fn { body, .. }) => {
1439-
if body.is_none() {
1441+
if body.is_none() && !self.is_sdylib_interface {
14401442
self.dcx().emit_err(errors::AssocFnWithoutBody {
14411443
span: item.span,
14421444
replace_span: self.ending_semi_or_hi(item.span),
@@ -1684,6 +1686,7 @@ pub fn check_crate(
16841686
sess: &Session,
16851687
features: &Features,
16861688
krate: &Crate,
1689+
is_sdylib_interface: bool,
16871690
lints: &mut LintBuffer,
16881691
) -> bool {
16891692
let mut validator = AstValidator {
@@ -1695,6 +1698,7 @@ pub fn check_crate(
16951698
outer_impl_trait_span: None,
16961699
disallow_tilde_const: Some(TildeConstReason::Item),
16971700
extern_mod_safety: None,
1701+
is_sdylib_interface,
16981702
lint_buffer: lints,
16991703
};
17001704
visit::walk_crate(&mut validator, krate);

compiler/rustc_ast_pretty/src/pprust/mod.rs

+3-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ use std::borrow::Cow;
77
use rustc_ast as ast;
88
use rustc_ast::token::{Token, TokenKind};
99
use rustc_ast::tokenstream::{TokenStream, TokenTree};
10-
pub use state::{AnnNode, Comments, PpAnn, PrintState, State, print_crate};
10+
pub use state::{
11+
AnnNode, Comments, PpAnn, PrintState, State, print_crate, print_crate_as_interface,
12+
};
1113

1214
/// Print the token kind precisely, without converting `$crate` into its respective crate name.
1315
pub fn token_kind_to_string(tok: &TokenKind) -> Cow<'static, str> {

compiler/rustc_ast_pretty/src/pprust/state.rs

+32-4
Original file line numberDiff line numberDiff line change
@@ -221,6 +221,7 @@ pub struct State<'a> {
221221
pub s: pp::Printer,
222222
comments: Option<Comments<'a>>,
223223
ann: &'a (dyn PpAnn + 'a),
224+
is_sdylib_interface: bool,
224225
}
225226

226227
const INDENT_UNIT: isize = 4;
@@ -236,10 +237,39 @@ pub fn print_crate<'a>(
236237
is_expanded: bool,
237238
edition: Edition,
238239
g: &AttrIdGenerator,
240+
) -> String {
241+
let mut s = State {
242+
s: pp::Printer::new(),
243+
comments: Some(Comments::new(sm, filename, input)),
244+
ann,
245+
is_sdylib_interface: false,
246+
};
247+
248+
print_crate_inner(&mut s, krate, is_expanded, edition, g);
249+
250+
s.ann.post(&mut s, AnnNode::Crate(krate));
251+
s.s.eof()
252+
}
253+
254+
pub fn print_crate_as_interface(
255+
krate: &ast::Crate,
256+
edition: Edition,
257+
g: &AttrIdGenerator,
239258
) -> String {
240259
let mut s =
241-
State { s: pp::Printer::new(), comments: Some(Comments::new(sm, filename, input)), ann };
260+
State { s: pp::Printer::new(), comments: None, ann: &NoAnn, is_sdylib_interface: true };
242261

262+
print_crate_inner(&mut s, krate, false, edition, g);
263+
s.s.eof()
264+
}
265+
266+
fn print_crate_inner<'a>(
267+
s: &mut State<'a>,
268+
krate: &ast::Crate,
269+
is_expanded: bool,
270+
edition: Edition,
271+
g: &AttrIdGenerator,
272+
) {
243273
if is_expanded && !krate.attrs.iter().any(|attr| attr.has_name(sym::no_core)) {
244274
// We need to print `#![no_std]` (and its feature gate) so that
245275
// compiling pretty-printed source won't inject libstd again.
@@ -277,8 +307,6 @@ pub fn print_crate<'a>(
277307
s.print_item(item);
278308
}
279309
s.print_remaining_comments();
280-
s.ann.post(&mut s, AnnNode::Crate(krate));
281-
s.s.eof()
282310
}
283311

284312
/// Should two consecutive tokens be printed with a space between them?
@@ -1083,7 +1111,7 @@ impl<'a> PrintState<'a> for State<'a> {
10831111

10841112
impl<'a> State<'a> {
10851113
pub fn new() -> State<'a> {
1086-
State { s: pp::Printer::new(), comments: None, ann: &NoAnn }
1114+
State { s: pp::Printer::new(), comments: None, ann: &NoAnn, is_sdylib_interface: false }
10871115
}
10881116

10891117
fn commasep_cmnt<T, F, G>(&mut self, b: Breaks, elts: &[T], mut op: F, mut get_span: G)

compiler/rustc_ast_pretty/src/pprust/state/item.rs

+10
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,10 @@ impl<'a> State<'a> {
159159

160160
/// Pretty-prints an item.
161161
pub(crate) fn print_item(&mut self, item: &ast::Item) {
162+
if self.is_sdylib_interface && item.span.is_dummy() {
163+
// Do not print prelude for interface files.
164+
return;
165+
}
162166
self.hardbreak_if_not_bol();
163167
self.maybe_print_comment(item.span.lo());
164168
self.print_outer_attributes(&item.attrs);
@@ -693,6 +697,12 @@ impl<'a> State<'a> {
693697
self.nbsp();
694698
self.print_contract(contract);
695699
}
700+
if self.is_sdylib_interface {
701+
self.word(";");
702+
self.end(); // end inner head-block
703+
self.end(); // end outer head-block
704+
return;
705+
}
696706
if let Some(body) = body {
697707
self.nbsp();
698708
self.print_block_with_attrs(body, attrs);

compiler/rustc_codegen_gcc/src/back/lto.rs

+5-1
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,11 @@ use crate::{GccCodegenBackend, GccContext, SyncContext, to_gcc_opt_level};
4444

4545
pub fn crate_type_allows_lto(crate_type: CrateType) -> bool {
4646
match crate_type {
47-
CrateType::Executable | CrateType::Dylib | CrateType::Staticlib | CrateType::Cdylib => true,
47+
CrateType::Executable
48+
| CrateType::Dylib
49+
| CrateType::Staticlib
50+
| CrateType::Cdylib
51+
| CrateType::Sdylib => true,
4852
CrateType::Rlib | CrateType::ProcMacro => false,
4953
}
5054
}

compiler/rustc_codegen_llvm/src/back/lto.rs

+2-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,8 @@ fn crate_type_allows_lto(crate_type: CrateType) -> bool {
4141
| CrateType::Dylib
4242
| CrateType::Staticlib
4343
| CrateType::Cdylib
44-
| CrateType::ProcMacro => true,
44+
| CrateType::ProcMacro
45+
| CrateType::Sdylib => true,
4546
CrateType::Rlib => false,
4647
}
4748
}

compiler/rustc_codegen_llvm/src/debuginfo/gdb.rs

+5-1
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,11 @@ pub(crate) fn needs_gdb_debug_scripts_section(cx: &CodegenCx<'_, '_>) -> bool {
9595
// in the `.debug_gdb_scripts` section. For that reason, we make sure that the
9696
// section is only emitted for leaf crates.
9797
let embed_visualizers = cx.tcx.crate_types().iter().any(|&crate_type| match crate_type {
98-
CrateType::Executable | CrateType::Dylib | CrateType::Cdylib | CrateType::Staticlib => {
98+
CrateType::Executable
99+
| CrateType::Dylib
100+
| CrateType::Cdylib
101+
| CrateType::Staticlib
102+
| CrateType::Sdylib => {
99103
// These are crate types for which we will embed pretty printers since they
100104
// are treated as leaf crates.
101105
true

compiler/rustc_codegen_ssa/src/back/link.rs

+9-5
Original file line numberDiff line numberDiff line change
@@ -1054,9 +1054,10 @@ fn link_natively(
10541054
strip_with_external_utility(sess, stripcmd, out_filename, &["--strip-debug"])
10551055
}
10561056
// Per the manpage, `-x` is the maximum safe strip level for dynamic libraries. (#93988)
1057-
(Strip::Symbols, CrateType::Dylib | CrateType::Cdylib | CrateType::ProcMacro) => {
1058-
strip_with_external_utility(sess, stripcmd, out_filename, &["-x"])
1059-
}
1057+
(
1058+
Strip::Symbols,
1059+
CrateType::Dylib | CrateType::Cdylib | CrateType::ProcMacro | CrateType::Sdylib,
1060+
) => strip_with_external_utility(sess, stripcmd, out_filename, &["-x"]),
10601061
(Strip::Symbols, _) => {
10611062
strip_with_external_utility(sess, stripcmd, out_filename, &["--strip-all"])
10621063
}
@@ -1244,8 +1245,10 @@ fn add_sanitizer_libraries(
12441245
// which should be linked to both executables and dynamic libraries.
12451246
// Everywhere else the runtimes are currently distributed as static
12461247
// libraries which should be linked to executables only.
1247-
if matches!(crate_type, CrateType::Dylib | CrateType::Cdylib | CrateType::ProcMacro)
1248-
&& !(sess.target.is_like_darwin || sess.target.is_like_msvc)
1248+
if matches!(
1249+
crate_type,
1250+
CrateType::Dylib | CrateType::Cdylib | CrateType::ProcMacro | CrateType::Sdylib
1251+
) && !(sess.target.is_like_darwin || sess.target.is_like_msvc)
12491252
{
12501253
return;
12511254
}
@@ -1939,6 +1942,7 @@ fn add_late_link_args(
19391942
codegen_results: &CodegenResults,
19401943
) {
19411944
let any_dynamic_crate = crate_type == CrateType::Dylib
1945+
|| crate_type == CrateType::Sdylib
19421946
|| codegen_results.crate_info.dependency_formats.iter().any(|(ty, list)| {
19431947
*ty == crate_type && list.iter().any(|&linkage| linkage == Linkage::Dynamic)
19441948
});

compiler/rustc_codegen_ssa/src/back/linker.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1815,7 +1815,7 @@ pub(crate) fn linked_symbols(
18151815
crate_type: CrateType,
18161816
) -> Vec<(String, SymbolExportKind)> {
18171817
match crate_type {
1818-
CrateType::Executable | CrateType::Cdylib | CrateType::Dylib => (),
1818+
CrateType::Executable | CrateType::Cdylib | CrateType::Dylib | CrateType::Sdylib => (),
18191819
CrateType::Staticlib | CrateType::ProcMacro | CrateType::Rlib => {
18201820
return Vec::new();
18211821
}

compiler/rustc_codegen_ssa/src/back/symbol_export.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ fn crate_export_threshold(crate_type: CrateType) -> SymbolExportLevel {
2929
CrateType::Executable | CrateType::Staticlib | CrateType::ProcMacro | CrateType::Cdylib => {
3030
SymbolExportLevel::C
3131
}
32-
CrateType::Rlib | CrateType::Dylib => SymbolExportLevel::Rust,
32+
CrateType::Rlib | CrateType::Dylib | CrateType::Sdylib => SymbolExportLevel::Rust,
3333
}
3434
}
3535

@@ -45,7 +45,7 @@ pub fn crates_export_threshold(crate_types: &[CrateType]) -> SymbolExportLevel {
4545
}
4646

4747
fn reachable_non_generics_provider(tcx: TyCtxt<'_>, _: LocalCrate) -> DefIdMap<SymbolExportInfo> {
48-
if !tcx.sess.opts.output_types.should_codegen() {
48+
if !tcx.sess.opts.output_types.should_codegen() && !tcx.is_sdylib_interface_build() {
4949
return Default::default();
5050
}
5151

@@ -168,7 +168,7 @@ fn exported_symbols_provider_local(
168168
tcx: TyCtxt<'_>,
169169
_: LocalCrate,
170170
) -> &[(ExportedSymbol<'_>, SymbolExportInfo)] {
171-
if !tcx.sess.opts.output_types.should_codegen() {
171+
if !tcx.sess.opts.output_types.should_codegen() && !tcx.is_sdylib_interface_build() {
172172
return &[];
173173
}
174174

compiler/rustc_codegen_ssa/src/base.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1027,7 +1027,7 @@ impl CrateInfo {
10271027
}
10281028

10291029
let embed_visualizers = tcx.crate_types().iter().any(|&crate_type| match crate_type {
1030-
CrateType::Executable | CrateType::Dylib | CrateType::Cdylib => {
1030+
CrateType::Executable | CrateType::Dylib | CrateType::Cdylib | CrateType::Sdylib => {
10311031
// These are crate types for which we invoke the linker and can embed
10321032
// NatVis visualizers.
10331033
true

compiler/rustc_driver_impl/src/lib.rs

+5-2
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@ use rustc_metadata::locator;
5454
use rustc_middle::ty::TyCtxt;
5555
use rustc_parse::{new_parser_from_file, new_parser_from_source_str, unwrap_or_emit_fatal};
5656
use rustc_session::config::{
57-
CG_OPTIONS, ErrorOutputType, Input, OptionDesc, OutFileName, OutputType, UnstableOptions,
58-
Z_OPTIONS, nightly_options, parse_target_triple,
57+
CG_OPTIONS, CrateType, ErrorOutputType, Input, OptionDesc, OutFileName, OutputType,
58+
UnstableOptions, Z_OPTIONS, nightly_options, parse_target_triple,
5959
};
6060
use rustc_session::getopts::{self, Matches};
6161
use rustc_session::lint::{Lint, LintId};
@@ -352,6 +352,8 @@ pub fn run_compiler(at_args: &[String], callbacks: &mut (dyn Callbacks + Send))
352352

353353
passes::write_dep_info(tcx);
354354

355+
passes::write_interface(tcx);
356+
355357
if sess.opts.output_types.contains_key(&OutputType::DepInfo)
356358
&& sess.opts.output_types.len() == 1
357359
{
@@ -816,6 +818,7 @@ fn print_crate_info(
816818
let supported_crate_types = CRATE_TYPES
817819
.iter()
818820
.filter(|(_, crate_type)| !invalid_output_for_target(&sess, *crate_type))
821+
.filter(|(_, crate_type)| *crate_type != CrateType::Sdylib)
819822
.map(|(crate_type_sym, _)| *crate_type_sym)
820823
.collect::<BTreeSet<_>>();
821824
for supported_crate_type in supported_crate_types {

compiler/rustc_feature/src/builtin_attrs.rs

+5
Original file line numberDiff line numberDiff line change
@@ -521,6 +521,11 @@ pub static BUILTIN_ATTRIBUTES: &[BuiltinAttribute] = &[
521521
naked_functions, experimental!(naked)
522522
),
523523

524+
gated!(
525+
export, Normal, template!(Word), WarnFollowing,
526+
EncodeCrossCrate::No, experimental!(export)
527+
),
528+
524529
// Testing:
525530
gated!(
526531
test_runner, CrateLevel, template!(List: "path"), ErrorFollowing,

compiler/rustc_feature/src/unstable.rs

+2
Original file line numberDiff line numberDiff line change
@@ -479,6 +479,8 @@ declare_features! (
479479
(unstable, explicit_extern_abis, "CURRENT_RUSTC_VERSION", Some(134986)),
480480
/// Allows explicit tail calls via `become` expression.
481481
(incomplete, explicit_tail_calls, "1.72.0", Some(112788)),
482+
/// Allows using `#[export]` which indicates that an item is exportable.
483+
(incomplete, export, "CURRENT_RUSTC_VERSION", Some(139939)),
482484
/// Allows using `aapcs`, `efiapi`, `sysv64` and `win64` as calling conventions
483485
/// for functions with varargs.
484486
(unstable, extended_varargs_abi_support, "1.65.0", Some(100189)),

compiler/rustc_interface/src/passes.rs

+27-1
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,11 @@ use rustc_resolve::Resolver;
3030
use rustc_session::config::{CrateType, Input, OutFileName, OutputFilenames, OutputType};
3131
use rustc_session::cstore::Untracked;
3232
use rustc_session::output::{collect_crate_types, filename_for_input};
33+
use rustc_session::parse::feature_err;
3334
use rustc_session::search_paths::PathKind;
3435
use rustc_session::{Limit, Session};
3536
use rustc_span::{
36-
ErrorGuaranteed, FileName, SourceFileHash, SourceFileHashAlgorithm, Span, Symbol, sym,
37+
DUMMY_SP, ErrorGuaranteed, FileName, SourceFileHash, SourceFileHashAlgorithm, Span, Symbol, sym,
3738
};
3839
use rustc_target::spec::PanicStrategy;
3940
use rustc_trait_selection::traits;
@@ -236,6 +237,7 @@ fn configure_and_expand(
236237
sess,
237238
features,
238239
&krate,
240+
tcx.is_sdylib_interface_build(),
239241
resolver.lint_buffer(),
240242
)
241243
});
@@ -252,6 +254,9 @@ fn configure_and_expand(
252254
sess.dcx().emit_err(errors::MixedProcMacroCrate);
253255
}
254256
}
257+
if crate_types.contains(&CrateType::Sdylib) && !tcx.features().export() {
258+
feature_err(sess, sym::export, DUMMY_SP, "`sdylib` crate type is unstable").emit();
259+
}
255260

256261
if is_proc_macro_crate && sess.panic_strategy() == PanicStrategy::Abort {
257262
sess.dcx().emit_warn(errors::ProcMacroCratePanicAbort);
@@ -741,6 +746,25 @@ pub fn write_dep_info(tcx: TyCtxt<'_>) {
741746
}
742747
}
743748

749+
pub fn write_interface<'tcx>(tcx: TyCtxt<'tcx>) {
750+
if !tcx.crate_types().contains(&rustc_session::config::CrateType::Sdylib) {
751+
return;
752+
}
753+
let _timer = tcx.sess.timer("write_interface");
754+
let (_, krate) = &*tcx.resolver_for_lowering().borrow();
755+
756+
let krate = rustc_ast_pretty::pprust::print_crate_as_interface(
757+
krate,
758+
tcx.sess.psess.edition,
759+
&tcx.sess.psess.attr_id_generator,
760+
);
761+
let export_output = tcx.output_filenames(()).interface_path();
762+
let mut file = fs::File::create_buffered(export_output).unwrap();
763+
if let Err(err) = write!(file, "{}", krate) {
764+
tcx.dcx().fatal(format!("error writing interface file: {}", err));
765+
}
766+
}
767+
744768
pub static DEFAULT_QUERY_PROVIDERS: LazyLock<Providers> = LazyLock::new(|| {
745769
let providers = &mut Providers::default();
746770
providers.analysis = analysis;
@@ -913,6 +937,8 @@ fn run_required_analyses(tcx: TyCtxt<'_>) {
913937
CStore::from_tcx(tcx).report_unused_deps(tcx);
914938
},
915939
{
940+
tcx.ensure_ok().exportable_items(LOCAL_CRATE);
941+
tcx.ensure_ok().stable_order_of_exportable_impls(LOCAL_CRATE);
916942
tcx.par_hir_for_each_module(|module| {
917943
tcx.ensure_ok().check_mod_loops(module);
918944
tcx.ensure_ok().check_mod_attrs(module);

0 commit comments

Comments
 (0)