diff options
| author | Ralf Jung <post@ralfj.de> | 2024-03-21 13:53:00 +0100 |
|---|---|---|
| committer | Ralf Jung <post@ralfj.de> | 2024-04-23 23:02:54 +0200 |
| commit | 173d1bd36be1925c0cb0b0bf55740c26db4ac476 (patch) | |
| tree | a4ecdef1b76bd546deb5d3346ad378310c62868b /compiler/rustc_const_eval/src/interpret | |
| parent | bf021ea6258bcd886fae523e8448f3697dd041fe (diff) | |
| download | rust-173d1bd36be1925c0cb0b0bf55740c26db4ac476.tar.gz rust-173d1bd36be1925c0cb0b0bf55740c26db4ac476.zip | |
properly fill a promoted's required_consts
then we can also make all_required_consts_are_checked a constant instead of a function
Diffstat (limited to 'compiler/rustc_const_eval/src/interpret')
| -rw-r--r-- | compiler/rustc_const_eval/src/interpret/eval_context.rs | 4 | ||||
| -rw-r--r-- | compiler/rustc_const_eval/src/interpret/machine.rs | 2 |
2 files changed, 2 insertions, 4 deletions
diff --git a/compiler/rustc_const_eval/src/interpret/eval_context.rs b/compiler/rustc_const_eval/src/interpret/eval_context.rs index 8f7c2580ad4..126d64329f8 100644 --- a/compiler/rustc_const_eval/src/interpret/eval_context.rs +++ b/compiler/rustc_const_eval/src/interpret/eval_context.rs @@ -1179,9 +1179,7 @@ impl<'mir, 'tcx: 'mir, M: Machine<'mir, 'tcx>> InterpCx<'mir, 'tcx, M> { ) -> InterpResult<'tcx, OpTy<'tcx, M::Provenance>> { M::eval_mir_constant(self, *val, span, layout, |ecx, val, span, layout| { let const_val = val.eval(*ecx.tcx, ecx.param_env, span).map_err(|err| { - if M::all_required_consts_are_checked(self) - && !matches!(err, ErrorHandled::TooGeneric(..)) - { + if M::ALL_CONSTS_ARE_PRECHECKED && !matches!(err, ErrorHandled::TooGeneric(..)) { // Looks like the const is not captued by `required_consts`, that's bad. bug!("interpret const eval failure of {val:?} which is not in required_consts"); } diff --git a/compiler/rustc_const_eval/src/interpret/machine.rs b/compiler/rustc_const_eval/src/interpret/machine.rs index e8ecbe089c3..8bc569bed54 100644 --- a/compiler/rustc_const_eval/src/interpret/machine.rs +++ b/compiler/rustc_const_eval/src/interpret/machine.rs @@ -142,7 +142,7 @@ pub trait Machine<'mir, 'tcx: 'mir>: Sized { /// Determines whether `eval_mir_constant` can never fail because all required consts have /// already been checked before. - fn all_required_consts_are_checked(ecx: &InterpCx<'mir, 'tcx, Self>) -> bool; + const ALL_CONSTS_ARE_PRECHECKED: bool = true; /// Whether memory accesses should be alignment-checked. fn enforce_alignment(ecx: &InterpCx<'mir, 'tcx, Self>) -> bool; |
