Verify that spans point to char boundaries

This makes invalid spans a lot easier to debug. A quick and unscientific
benchmarking test revealed that the performance impact of this is small
at most.

Author: Noratrieb

compiler/rustc_span/src/lib.rs

@@ -123,18 +123,15 @@ pub fn set_session_globals_then<R>(session_globals: &SessionGlobals, f: impl FnO
 }
 
 #[inline]
-pub fn create_default_session_if_not_set_then<R, F>(f: F) -> R
-where
-    F: FnOnce(&SessionGlobals) -> R,
-{
+pub fn create_default_session_if_not_set_then<R>(f: impl FnOnce(&SessionGlobals) -> R) -> R {
     create_session_if_not_set_then(edition::DEFAULT_EDITION, f)
 }
 
 #[inline]
-pub fn create_session_if_not_set_then<R, F>(edition: Edition, f: F) -> R
-where
-    F: FnOnce(&SessionGlobals) -> R,
-{
+pub fn create_session_if_not_set_then<R>(
+    edition: Edition,
+    f: impl FnOnce(&SessionGlobals) -> R,
+) -> R {
     if !SESSION_GLOBALS.is_set() {
         let session_globals = SessionGlobals::new(edition);
         SESSION_GLOBALS.set(&session_globals, || SESSION_GLOBALS.with(f))
@@ -151,6 +148,13 @@ where
     SESSION_GLOBALS.with(f)
 }
 
+pub struct NotSet;
+
+#[inline]
+pub fn try_with_session_globals<R>(f: impl FnOnce(&SessionGlobals) -> R) -> Result<R, NotSet> {
+    if SESSION_GLOBALS.is_set() { Ok(SESSION_GLOBALS.with(f)) } else { Err(NotSet) }
+}
+
 #[inline]
 pub fn create_default_session_globals_then<R>(f: impl FnOnce() -> R) -> R {
     create_session_globals_then(edition::DEFAULT_EDITION, f)

compiler/rustc_span/src/span_encoding.rs

@@ -6,7 +6,7 @@
 
 use crate::def_id::{DefIndex, LocalDefId};
 use crate::hygiene::SyntaxContext;
-use crate::SPAN_TRACK;
+use crate::{try_with_session_globals, NotSet, Pos, SPAN_TRACK};
 use crate::{BytePos, SpanData};
 
 use rustc_data_structures::fx::FxIndexSet;
@@ -96,6 +96,40 @@ impl Span {
         ctxt: SyntaxContext,
         parent: Option<LocalDefId>,
     ) -> Self {
+        // Make sure that the byte positions are at char boundaries.
+        // Only do this if the session globals are set correctly, which they aren't in some unit tests.
+        if cfg!(debug_assertions) {
+            let _: Result<(), NotSet> = try_with_session_globals(|sess| {
+                let sm = sess.source_map.lock();
+                if let Some(sm) = &*sm {
+                    let offset = sm.lookup_byte_offset(lo);
+                    if let Some(file) = &offset.sf.src {
+                        // `is_char_boundary` already checks this, but asserting it seperately gives a better panic message.
+                        assert!(
+                            file.len() >= offset.pos.to_usize(),
+                            "start of span is out of bounds"
+                        );
+                        assert!(
+                            file.is_char_boundary(offset.pos.to_usize()),
+                            "start of span not on char boundary"
+                        );
+                    }
+
+                    let offset = sm.lookup_byte_offset(hi);
+                    if let Some(file) = &offset.sf.src {
+                        assert!(
+                            file.len() >= offset.pos.to_usize(),
+                            "end of span is out of bounds"
+                        );
+                        assert!(
+                            file.is_char_boundary(offset.pos.to_usize()),
+                            "end of span not on char boundary"
+                        );
+                    }
+                }
+            });
+        }
+
         if lo > hi {
             std::mem::swap(&mut lo, &mut hi);
         }