Inline pinned_borrows access

Remove the now-unnecessary helper method and access state.pinned_borrows directly.

Author: P8L1

compiler/rustc_borrowck/src/lib.rs

@@ -1270,13 +1270,6 @@ impl<'a, 'tcx> MirBorrowckCtxt<'a, '_, 'tcx> {
         }
     }
 
-    fn pinned_borrows<'s>(
-        &self,
-        state: &'s BorrowckDomain,
-    ) -> Option<&'s MixedBitSet<BorrowIndex>> {
-        Some(&state.pinned_borrows)
-    }
-
     #[instrument(level = "debug", skip(self, state))]
     fn check_access_for_conflict(
         &mut self,
@@ -1290,7 +1283,7 @@ impl<'a, 'tcx> MirBorrowckCtxt<'a, '_, 'tcx> {
 
         let borrows_in_scope = self.borrows_in_scope(location, state);
         debug!(?borrows_in_scope, ?location);
-        let pinned_borrows = self.pinned_borrows(state);
+        let pinned_borrows = &state.pinned_borrows;
 
         each_borrow_involving_path(
             self,
@@ -1299,8 +1292,7 @@ impl<'a, 'tcx> MirBorrowckCtxt<'a, '_, 'tcx> {
             (sd, place_span.0),
             self.borrow_set,
             |borrow_index| {
-                borrows_in_scope.contains(borrow_index)
-                    || pinned_borrows.is_some_and(|p| p.contains(borrow_index))
+                borrows_in_scope.contains(borrow_index) || pinned_borrows.contains(borrow_index)
             },
             |this, borrow_index, borrow| match (rw, borrow.kind) {
                 // Obviously an activation is compatible with its own
@@ -1383,7 +1375,7 @@ impl<'a, 'tcx> MirBorrowckCtxt<'a, '_, 'tcx> {
                     if !borrows_in_scope.contains(borrow_index) =>
                 {
                     debug_assert!(
-                        pinned_borrows.is_none_or(|p| p.contains(borrow_index)),
+                        pinned_borrows.contains(borrow_index),
                         "unexpected expired but non-pinned borrow {borrow_index:?}: {borrow:?}",
                     );
                     match kind {