Skip to content
This repository has been archived by the owner. It is now read-only.

Parsing text based radeco-ir into graph representation #117

Closed
sushant94 opened this issue Jan 23, 2018 · 3 comments · Fixed by #126
Closed

Parsing text based radeco-ir into graph representation #117

sushant94 opened this issue Jan 23, 2018 · 3 comments · Fixed by #126

Comments

@sushant94
Copy link
Collaborator

@sushant94 sushant94 commented Jan 23, 2018

This would allow us to write IR to files and load them up at a later point in the analysis. It would be nice if we could do this with a parser generator in rust, such as lalrpop

@HMPerson1

This comment has been minimized.

Copy link
Collaborator

@HMPerson1 HMPerson1 commented Feb 22, 2018

I'd like to work on this.

So essentially I would write a function

fn parse_il(s: &str) -> SSAStorage

correct?

@HMPerson1

This comment has been minimized.

Copy link
Collaborator

@HMPerson1 HMPerson1 commented Feb 22, 2018

It also looks like a lot of information is dropped in emit_il. Should I also change it so that the IR can be round-tripped through its text representation?

@sushant94

This comment has been minimized.

Copy link
Collaborator Author

@sushant94 sushant94 commented Feb 22, 2018

@HMPerson1 Yes, that is the plan. I'd suggest making a simple version of it first, test it out. Then go top-down and add in missing information, enhance it, etc.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants
You can’t perform that action at this time.