Skip to content

Call get_language_options from get_goto_model #588

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.

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

Merged
merged 1 commit into from
Apr 4, 2017

Conversation

smowton
Copy link
Contributor

@smowton smowton commented Mar 3, 2017

get_language_options is called from cbmc_parse_options. It should be called by other driver programs (which use get-goto-model) too; otherwise the Java frontend's (e.g.) skip-runtime-checks parameter remains uninitialised.

@smowton smowton force-pushed the sss_get_language_options branch 2 times, most recently from 7f00150 to d9dd394 Compare March 3, 2017 14:22
@smowton smowton force-pushed the sss_get_language_options branch 2 times, most recently from 9bfec7f to edc2236 Compare March 8, 2017 09:14
Copy link
Collaborator

@tautschnig tautschnig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR appears to be doing a whole lot of things that I'm failing to relate to "Call get_language_options from get_goto_model" !?

@smowton smowton force-pushed the sss_get_language_options branch from edc2236 to 32515a9 Compare March 10, 2017 10:11
@smowton
Copy link
Contributor Author

smowton commented Mar 10, 2017

Updated (source and target branches had diverged more since this was submitted)

@tautschnig
Copy link
Collaborator

@smowton Thanks a lot for the cleanup! Would you mind adding the "WHY" to the commit message - it just states the "WHAT".

@smowton smowton force-pushed the sss_get_language_options branch from 32515a9 to 98ec7bb Compare March 16, 2017 11:32
This allows language frontends to consume command-line options,
which were previously left uninitialised when using goto-analyzer
or symex frontends.
@smowton smowton force-pushed the sss_get_language_options branch from 98ec7bb to 7371265 Compare March 16, 2017 11:34
@smowton
Copy link
Contributor Author

smowton commented Mar 16, 2017

@tautschnig done, apologies hadn't noticed this

@smowton
Copy link
Contributor Author

smowton commented Mar 31, 2017

Any reason this is stuck?

@kroening kroening merged commit bc483df into diffblue:master Apr 4, 2017
NathanJPhillips pushed a commit to NathanJPhillips/cbmc that referenced this pull request Sep 6, 2018
…itive_rebuild

--rebuild should change function depending on context of run
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants