Auto merge of #71775 - petrochenkov:crtcfg, r=matthewjasper

Enable `cfg` predicate for `target_feature = "crt-static"` only if the target supports it

That's what all other `target_feature`s do.
This commit is contained in:
bors 2020-05-10 08:25:32 +00:00
commit 8d16eeb8c9
3 changed files with 6 additions and 14 deletions

View file

@ -49,7 +49,7 @@ pub fn add_configuration(
cfg.extend(codegen_backend.target_features(sess).into_iter().map(|feat| (tf, Some(feat))));
if sess.crt_static_feature(None) {
if sess.crt_static(None) {
cfg.insert((tf, Some(Symbol::intern("crt-static"))));
}
}

View file

@ -598,16 +598,11 @@ impl Session {
/// Check whether this compile session and crate type use static crt.
pub fn crt_static(&self, crate_type: Option<CrateType>) -> bool {
// If the target does not opt in to crt-static support, use its default.
if self.target.target.options.crt_static_respected {
self.crt_static_feature(crate_type)
} else {
self.target.target.options.crt_static_default
if !self.target.target.options.crt_static_respected {
// If the target does not opt in to crt-static support, use its default.
return self.target.target.options.crt_static_default;
}
}
/// Check whether this compile session and crate type use `crt-static` feature.
pub fn crt_static_feature(&self, crate_type: Option<CrateType>) -> bool {
let requested_features = self.opts.cg.target_feature.split(',');
let found_negative = requested_features.clone().any(|r| r == "-crt-static");
let found_positive = requested_features.clone().any(|r| r == "+crt-static");

View file

@ -1,9 +1,6 @@
// run-pass
#![allow(stable_features)]
// compile-flags:-C target-feature=+crt-static -Z unstable-options
#![feature(cfg_target_feature)]
// compile-flags:-C target-feature=+crt-static
// only-msvc
#[cfg(target_feature = "crt-static")]
fn main() {}