Skip to content

Commit

Permalink
Make allow_internal_unstable work with stmt_expr_attributes
Browse files Browse the repository at this point in the history
  • Loading branch information
Jules-Bertholet committed Oct 31, 2023
1 parent a395214 commit 42b1406
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 5 deletions.
4 changes: 3 additions & 1 deletion compiler/rustc_expand/src/config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -449,7 +449,9 @@ impl<'a> StripUnconfigured<'a> {
/// If attributes are not allowed on expressions, emit an error for `attr`
#[instrument(level = "trace", skip(self))]
pub(crate) fn maybe_emit_expr_attr_err(&self, attr: &Attribute) {
if self.features.is_some_and(|features| !features.stmt_expr_attributes) {
if self.features.is_some_and(|features| !features.stmt_expr_attributes)
&& !attr.span.allows_unstable(sym::stmt_expr_attributes)
{
let mut err = feature_err(
&self.sess.parse_sess,
sym::stmt_expr_attributes,
Expand Down
10 changes: 10 additions & 0 deletions tests/ui/internal/internal-unstable.rs
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,14 @@ macro_rules! bar {
}}
}

#[allow_internal_unstable(stmt_expr_attributes)]
macro_rules! internal_attr {
($e: expr) => {
#[allow(overflowing_literals)]
$e
}
}

fn main() {
// ok, the instability is contained.
call_unstable_allow!();
Expand All @@ -51,4 +59,6 @@ fn main() {
#[allow_internal_unstable]
_ => {}
}

assert_eq!(internal_attr!(1e100_f32), f32::INFINITY);
}
8 changes: 4 additions & 4 deletions tests/ui/internal/internal-unstable.stderr
Original file line number Diff line number Diff line change
@@ -1,29 +1,29 @@
error[E0658]: use of unstable library feature 'function'
--> $DIR/internal-unstable.rs:40:25
--> $DIR/internal-unstable.rs:48:25
|
LL | pass_through_allow!(internal_unstable::unstable());
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: add `#![feature(function)]` to the crate attributes to enable

error[E0658]: use of unstable library feature 'function'
--> $DIR/internal-unstable.rs:42:27
--> $DIR/internal-unstable.rs:50:27
|
LL | pass_through_noallow!(internal_unstable::unstable());
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: add `#![feature(function)]` to the crate attributes to enable

error[E0658]: use of unstable library feature 'function'
--> $DIR/internal-unstable.rs:46:22
--> $DIR/internal-unstable.rs:54:22
|
LL | println!("{:?}", internal_unstable::unstable());
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: add `#![feature(function)]` to the crate attributes to enable

error[E0658]: use of unstable library feature 'function'
--> $DIR/internal-unstable.rs:48:10
--> $DIR/internal-unstable.rs:56:10
|
LL | bar!(internal_unstable::unstable());
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
Expand Down

0 comments on commit 42b1406

Please sign in to comment.