Skip to content

Commit

Permalink
convert Rvalue::threadlocalref assertion to delay bug
Browse files Browse the repository at this point in the history
  • Loading branch information
csmoe committed Jun 2, 2021
1 parent 9283956 commit 521d9ab
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 17 deletions.
13 changes: 4 additions & 9 deletions compiler/rustc_mir/src/transform/check_consts/validation.rs
Original file line number Diff line number Diff line change
Expand Up @@ -356,10 +356,9 @@ impl Validator<'mir, 'tcx> {
}

fn check_static(&mut self, def_id: DefId, span: Span) {
assert!(
!self.tcx.is_thread_local_static(def_id),
"tls access is checked in `Rvalue::ThreadLocalRef"
);
if self.tcx.is_thread_local_static(def_id) {
self.tcx.sess.delay_span_bug(span, "tls access is checked in `Rvalue::ThreadLocalRef");
}
self.check_op_spanned(ops::StaticAccess, span)
}

Expand Down Expand Up @@ -732,11 +731,7 @@ impl Visitor<'tcx> for Validator<'mir, 'tcx> {
if proj_base.is_empty() {
if let (local, []) = (place_local, proj_base) {
let decl = &self.body.local_decls[local];
if let Some(box LocalInfo::StaticRef {
def_id,
is_thread_local: false,
}) = decl.local_info
{
if let Some(box LocalInfo::StaticRef { def_id, .. }) = decl.local_info {
let span = decl.source_info.span;
self.check_static(def_id, span);
return;
Expand Down
2 changes: 1 addition & 1 deletion src/test/ui/thread-local-static.rs
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@ const fn g(x: &mut [u32; 8]) {
//~^ ERROR mutable references are not allowed
std::mem::swap(x, &mut STATIC_VAR_2)
//~^ ERROR thread-local statics cannot be accessed
//~| ERROR dereferencing raw pointers in constant
//~| ERROR mutable references are not allowed
//~| ERROR use of mutable static is unsafe
//~| constant functions cannot refer to statics
}

fn main() {}
13 changes: 6 additions & 7 deletions src/test/ui/thread-local-static.stderr
Original file line number Diff line number Diff line change
Expand Up @@ -13,14 +13,13 @@ error[E0625]: thread-local statics cannot be accessed at compile-time
LL | std::mem::swap(x, &mut STATIC_VAR_2)
| ^^^^^^^^^^^^

error[E0658]: dereferencing raw pointers in constant functions is unstable
--> $DIR/thread-local-static.rs:9:23
error[E0013]: constant functions cannot refer to statics
--> $DIR/thread-local-static.rs:9:28
|
LL | std::mem::swap(x, &mut STATIC_VAR_2)
| ^^^^^^^^^^^^^^^^^
| ^^^^^^^^^^^^
|
= note: see issue #51911 <https://github.com/rust-lang/rust/issues/51911> for more information
= help: add `#![feature(const_raw_ptr_deref)]` to the crate attributes to enable
= help: consider extracting the value of the `static` to a `const`, and referring to that

error[E0658]: mutable references are not allowed in constant functions
--> $DIR/thread-local-static.rs:9:23
Expand All @@ -41,5 +40,5 @@ LL | std::mem::swap(x, &mut STATIC_VAR_2)

error: aborting due to 5 previous errors

Some errors have detailed explanations: E0133, E0658.
For more information about an error, try `rustc --explain E0133`.
Some errors have detailed explanations: E0013, E0133, E0658.
For more information about an error, try `rustc --explain E0013`.

0 comments on commit 521d9ab

Please sign in to comment.