Skip to content

Commit

Permalink
Add tests for lint on type dependent on consts
Browse files Browse the repository at this point in the history
  • Loading branch information
JulianKnodt committed May 17, 2022
1 parent c1d65ea commit edae6ed
Show file tree
Hide file tree
Showing 5 changed files with 114 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,9 @@ pub fn is_const_evaluatable<'cx, 'tcx>(
}

let future_compat_lint = || {
if tcx.features().generic_const_exprs {
return;
}
if let Some(local_def_id) = uv.def.did.as_local() {
infcx.tcx.struct_span_lint_hir(
lint::builtin::CONST_EVALUATABLE_UNCHECKED,
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
error: generic parameters may not be used in const operations
--> $DIR/dependence_lint.rs:13:32
|
LL | let _: [u8; size_of::<*mut T>()]; // error on stable, error with gce
| ^ cannot perform const operation using `T`
|
= note: type parameters may not be used in const expressions
= help: use `#![feature(generic_const_exprs)]` to allow generic const expressions

error: generic parameters may not be used in const operations
--> $DIR/dependence_lint.rs:20:37
|
LL | let _: [u8; if true { size_of::<T>() } else { 3 }]; // error on stable, error with gce
| ^ cannot perform const operation using `T`
|
= note: type parameters may not be used in const expressions
= help: use `#![feature(generic_const_exprs)]` to allow generic const expressions

warning: cannot use constants which depend on generic parameters in types
--> $DIR/dependence_lint.rs:9:9
|
LL | [0; size_of::<*mut T>()]; // lint on stable, error with `generic_const_exprs`
| ^^^^^^^^^^^^^^^^^^^
|
= note: `#[warn(const_evaluatable_unchecked)]` on by default
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
= note: for more information, see issue #76200 <https://github.com/rust-lang/rust/issues/76200>

warning: cannot use constants which depend on generic parameters in types
--> $DIR/dependence_lint.rs:16:9
|
LL | [0; if false { size_of::<T>() } else { 3 }]; // lint on stable, error with gce
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
= note: for more information, see issue #76200 <https://github.com/rust-lang/rust/issues/76200>

error: aborting due to 2 previous errors; 2 warnings emitted

Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
error: overly complex generic constant
--> $DIR/dependence_lint.rs:16:9
|
LL | [0; if false { size_of::<T>() } else { 3 }]; // lint on stable, error with gce
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ control flow is not supported in generic constants
|
= help: consider moving this anonymous constant into a `const` function

error: overly complex generic constant
--> $DIR/dependence_lint.rs:20:17
|
LL | let _: [u8; if true { size_of::<T>() } else { 3 }]; // error on stable, error with gce
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ control flow is not supported in generic constants
|
= help: consider moving this anonymous constant into a `const` function

error: unconstrained generic constant
--> $DIR/dependence_lint.rs:13:12
|
LL | let _: [u8; size_of::<*mut T>()]; // error on stable, error with gce
| ^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: try adding a `where` bound using this expression: `where [(); size_of::<*mut T>()]:`

error: unconstrained generic constant
--> $DIR/dependence_lint.rs:9:9
|
LL | [0; size_of::<*mut T>()]; // lint on stable, error with `generic_const_exprs`
| ^^^^^^^^^^^^^^^^^^^
|
= help: try adding a `where` bound using this expression: `where [(); size_of::<*mut T>()]:`

error: aborting due to 4 previous errors

25 changes: 25 additions & 0 deletions src/test/ui/const-generics/generic_const_exprs/dependence_lint.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
// revisions: full gce

#![cfg_attr(gce, feature(generic_const_exprs))]
#![allow(incomplete_features)]

use std::mem::size_of;

fn foo<T>() {
[0; size_of::<*mut T>()]; // lint on stable, error with `generic_const_exprs`
//[gce]~^ ERROR unconstrained
//[full]~^^ WARNING cannot use constants
//[full]~| WARNING this was previously accepted
let _: [u8; size_of::<*mut T>()]; // error on stable, error with gce
//[full]~^ ERROR generic parameters may not be used
//[gce]~^^ ERROR unconstrained generic
[0; if false { size_of::<T>() } else { 3 }]; // lint on stable, error with gce
//[gce]~^ ERROR overly complex
//[full]~^^ WARNING cannot use constants
//[full]~| WARNING this was previously accepted
let _: [u8; if true { size_of::<T>() } else { 3 }]; // error on stable, error with gce
//[full]~^ ERROR generic parameters may not be used
//[gce]~^^ ERROR overly complex
}

fn main() {}
13 changes: 13 additions & 0 deletions src/test/ui/const-generics/generic_const_exprs/no_dependence.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// check-pass
#![feature(generic_const_exprs)]
#![allow(incomplete_features)]

fn two_args<const N: usize, const M: usize>() -> [u8; M + 2] {
[0; M + 2]
}

fn yay<const N: usize>() -> [u8; 4] {
two_args::<N, 2>() // no lint
}

fn main() {}

0 comments on commit edae6ed

Please sign in to comment.