Remove -Z strip.

It was stabilized as `-C strip` in November 2021. The unstable option
was kept around as a temporary measure to ease the transition. Two years
is more than enough!
This commit is contained in:
Nicholas Nethercote 2023-11-09 11:36:02 +11:00
parent fdaaaf9f92
commit ecc936b155
2 changed files with 2 additions and 12 deletions

View File

@ -1033,7 +1033,7 @@ fn is_illegal_instruction(_status: &ExitStatus) -> bool {
SplitDebuginfo::Packed => link_dwarf_object(sess, codegen_results, out_filename),
}
let strip = strip_value(sess);
let strip = sess.opts.cg.strip;
if sess.target.is_like_osx {
match (strip, crate_type) {
@ -1070,14 +1070,6 @@ fn is_illegal_instruction(_status: &ExitStatus) -> bool {
Ok(())
}
// Temporarily support both -Z strip and -C strip
fn strip_value(sess: &Session) -> Strip {
match (sess.opts.unstable_opts.strip, sess.opts.cg.strip) {
(s, Strip::None) => s,
(_, s) => s,
}
}
fn strip_symbols_with_external_utility<'a>(
sess: &'a Session,
util: &str,
@ -2370,7 +2362,7 @@ fn add_order_independent_options(
);
// Pass debuginfo, NatVis debugger visualizers and strip flags down to the linker.
cmd.debuginfo(strip_value(sess), &natvis_visualizers);
cmd.debuginfo(sess.opts.cg.strip, &natvis_visualizers);
// We want to prevent the compiler from accidentally leaking in any system libraries,
// so by default we tell linkers not to link to any default libraries.

View File

@ -1841,8 +1841,6 @@ pub(crate) fn parse_inlining_threshold(slot: &mut InliningThreshold, v: Option<&
"prefer dynamic linking to static linking for staticlibs (default: no)"),
strict_init_checks: bool = (false, parse_bool, [TRACKED],
"control if mem::uninitialized and mem::zeroed panic on more UB"),
strip: Strip = (Strip::None, parse_strip, [UNTRACKED],
"tell the linker which information to strip (`none` (default), `debuginfo` or `symbols`)"),
#[rustc_lint_opt_deny_field_access("use `Session::teach` instead of this field")]
teach: bool = (false, parse_bool, [TRACKED],
"show extended diagnostic help (default: no)"),