Skip to content

Make output of --dump-c --use-system-headers compilable #249

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
peterschrammel opened this issue Oct 7, 2016 · 2 comments
Closed

Make output of --dump-c --use-system-headers compilable #249

peterschrammel opened this issue Oct 7, 2016 · 2 comments
Assignees
@peterschrammel
Labels
C Front End Output task

Comments

@peterschrammel
Copy link
Member

In order to compile dumped C files with the original header files, we have to

  • retain typedefs involving anonymous structs/unions
  • prevent the output of structs/unions defined in system headers
  • prevent the output of built-ins
  • ...

A quick hack is here:
peterschrammel@ff6a7d1

More to come...
@peterschrammel peterschrammel self-assigned this Oct 7, 2016
@tautschnig
Copy link
Collaborator

@peterschrammel Has this been fully addressed by #858?

smowton pushed a commit to smowton/cbmc that referenced this issue May 9, 2018
@NathanJPhillips
Merge pull request diffblue#249 from diffblue/cleanup/catch-by-const-ref
2ef9b25 
Catch std::string by const ref
@tautschnig
Copy link
Collaborator

I'm inclined to close this issue as dump-c's output appears to work fine these days. It's certainly not perfect, but #5700 shows that it does generally work. I think we need concrete examples of failures to drive further fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@peterschrammel peterschrammel

Labels
C Front End Output task
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tautschnig @peterschrammel