Heapster ide info #1821
Merged
Heapster ide info #1821
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…nto heapster-ide-info
|
@scuellar it looks like the build is failing, due to some pattern-matching coverage warnings. Could you please take a shot at fixing these? |
eddywestbrook
added
the
subsystem: heapster
added 3 commits
February 28, 2023 16:59
…edMb.hs, to help with GHC 9 support
…nged the names of all of its operationsto use 'Named' as a suffix instead of just using a prime
eddywestbrook
approved these changes
Mar 1, 2023
There was a problem hiding this comment.
I worked closely with Karl and Eric M when they originally made this PR, so I'm pretty familiar with it. It adds some very useful functionality, specifically, the ability to export a JSON representation of a type-checked Crucible term, so that a UI can display that to the user.
|
Hmm, I'm now getting a seg fault on this branch on my local machine. Still debugging... |
eddywestbrook
added
the
PR: ready to merge
|
Never mind, it was a known issue with GHC 9.0. It works with GHC 8.10.4. So let's merge this sucker! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds the
heapster_dump_ide_infocommand to Heapster. The new instruction produces ajsonfile with Heapster permissions for each code block. The 'json' file can then be used by the VSCode extension to display the permissions on each block of source C code.This branch was developped a year ago and so the last handful of commits are bringing it up to date with master. We should merge it before it falls behind again.
The key files to look at are:
jsonJSONinstance declarations and the logic to export the permissions.Mbthat includes human readable names for variables, objects and code. Most of the changes are transformingMbinstances intoMb'instancesjsonfile and can be seen from the VSCode extension.