Skip to content

basic fault localization #181

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 3 commits into from
Aug 22, 2016
Merged

Conversation

peterschrammel
Copy link
Member

adds --localize-faults option
(and --localize-faults-method m)

with very simple scoring method as an example

This should be a start to serve as a framework for David to add further techniques.

class fault_localizationt
{
public:
fault_localizationt(const optionst &_options)
Copy link
Member

Choose a reason for hiding this comment

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

should be made "explicit"

@kroening kroening self-assigned this Jul 19, 2016
@peterschrammel
Copy link
Member Author

I have the feeling that there should be a bmc base class from which bmc_cover, bmc_all_properties and bmc_fault_localization should be derived...

@kroening
Copy link
Member

Yes, it's likely indeed time to split up the BMC implementation

@kroening kroening merged commit a986020 into diffblue:master Aug 22, 2016
@martin-cs
Copy link
Collaborator

On Mon, 2016-08-22 at 00:01 -0700, Daniel Kroening wrote:

Yes, it's likely indeed time to split up the BMC implementation

I tried doing this a few years ago. IIRC there isn't a 'low impact' way
of doing it.

Cheers,

  • Martin

smowton pushed a commit to smowton/cbmc that referenced this pull request May 9, 2018
…n_security_analyser

SEC-47: Tiny clean-up in security analyser.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants