CFG_RELEASE is mandatory, no need for option_env

dtolnay · dtolnay · commit ea298661680f · 2023-10-25T14:13:12.000-07:00
The same file already contains another env!("CFG_RELEASE") on line 611,
so it's impossible for this crate to compile without CFG_RELEASE set.
diff --git a/compiler/rustc_attr/src/builtin.rs b/compiler/rustc_attr/src/builtin.rs
@@ -24,8 +24,7 @@ use crate::session_diagnostics::{self, IncorrectReprFormatGenericCause};
 pub const VERSION_PLACEHOLDER: &str = "CURRENT_RUSTC_VERSION";
 
 pub fn rust_version_symbol() -> Symbol {
-    let version = option_env!("CFG_RELEASE").unwrap_or("<current>");
-    Symbol::intern(&version)
+    Symbol::intern(env!("CFG_RELEASE"))
 }
 
 pub fn is_builtin_attr(attr: &Attribute) -> bool {

Original file line number	Diff line number	Diff line change
`@@ -24,8 +24,7 @@ use crate::session_diagnostics::{self, IncorrectReprFormatGenericCause};`
`24`	`24`	`pub const VERSION_PLACEHOLDER: &str = "CURRENT_RUSTC_VERSION";`
`25`	`25`
`26`	`26`	`pub fn rust_version_symbol() -> Symbol {`
`27`		`- let version = option_env!("CFG_RELEASE").unwrap_or("<current>");`
`28`		`- Symbol::intern(&version)`
	`27`	`+ Symbol::intern(env!("CFG_RELEASE"))`
`29`	`28`	`}`
`30`	`29`
`31`	`30`	`pub fn is_builtin_attr(attr: &Attribute) -> bool {`