Skip to content

Commit

Permalink
crucible-mir: Cite #1223 for more info on CastKind limitations
Browse files Browse the repository at this point in the history
Currently, `crucible-mir`'s implementation of `CastKind` lags behind the
`rustc` API's `CastKind` enum. Let's leave a link to the relevant issue (#1223)
as breadcrumbs.
  • Loading branch information
RyanGlScott committed Jul 18, 2024
1 parent c310fcb commit 2909c22
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion crucible-mir/src/Mir/JSON.hs
Original file line number Diff line number Diff line change
Expand Up @@ -403,7 +403,8 @@ instance FromJSON CastKind where
Just (String "Pointer(MutToConstPointer)") -> pure MutToConstPointer
Just (String "UnsizeVtable") -> UnsizeVtable <$> v .: "vtable"
-- TODO: actually plumb this information through if it is relevant
-- instead of using Misc
-- instead of using Misc. See
-- https://github.com/GaloisInc/crucible/issues/1223
Just (String "PointerExposeAddress") -> pure Misc
Just (String "PointerFromExposedAddress") -> pure Misc
Just (String "DynStar") -> pure Misc
Expand Down
2 changes: 1 addition & 1 deletion crucible-mir/src/Mir/Mir.hs
Original file line number Diff line number Diff line change
Expand Up @@ -459,7 +459,7 @@ data Vtable = Vtable
}
deriving (Show, Eq, Ord, Generic)

-- TODO: add other castkinds (see json)
-- TODO: add other castkinds (see https://github.com/GaloisInc/crucible/issues/1223)
data CastKind =
Misc
| ReifyFnPointer
Expand Down

0 comments on commit 2909c22

Please sign in to comment.