Skip to content
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

Assertion failure--evaluation related? #104

Closed
yanniss opened this issue Apr 28, 2016 · 3 comments
Closed

Assertion failure--evaluation related? #104

yanniss opened this issue Apr 28, 2016 · 3 comments
Assignees
Labels
bug - identified Bugs with an identified cause

Comments

@yanniss
Copy link
Collaborator

yanniss commented Apr 28, 2016

I'm getting an assertion failure in the latest version:

souffle: RamTranslator.cpp:146: void souffle::{anonymous}::ValueIndex::addVarReference(const souffle::AstVariable&, const souffle::{anonymous}::Location&): Assertion `(locs.empty() || locs.back() < l) && "Unordered location insertion!"' failed.
Aborted (core dumped)

I have no clue what may be the cause and it will take me hours to construct a redacted example of the code that does the same computation. But here is the single-rule addition that precipitates the failure:

// If some access path is to be rebased, its prefixes are as well.
AccessPathShouldBeRebased( [ ?from, ?rest ], ?from, ?to) :-
AccessPathShouldBeRebased(?ap, ?from, ?to),
?ap = [ ?from, ?suffix ],
?suffix = [ ?rest, ?lastFld ].

Without this, the program performs meaningful computations so I have high confidence. The relation AccessPathShouldBeRebased is not populated anywhere else (though I tried that as well, and the failure persists) nor is it consumed anywhere at this point.

@yanniss yanniss added the bug - identified Bugs with an identified cause label Apr 28, 2016
@yanniss
Copy link
Collaborator Author

yanniss commented Apr 29, 2016

Here's a minimal example for the above. Just souffle this and it fails with the assertion, in either compiler or interpret mode. (I renamed the .dl file to .txt so github accepts it inline in the issue.) I've found no way to get around this problem, so it's currently stopping me from writing the analysis I intended. But perhaps there is a syntactic workaround I missed.
temp2.txt

@HerbertJordan
Copy link
Collaborator

Should be fixed by #106

@yanniss
Copy link
Collaborator Author

yanniss commented May 1, 2016

Thanks! I think it's indeed fixed. Will reopen if more complex tests reveal issues.

@yanniss yanniss closed this as completed May 1, 2016
b-scholz added a commit to b-scholz/souffle that referenced this issue May 22, 2016
b-scholz added a commit that referenced this issue May 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug - identified Bugs with an identified cause
Projects
None yet
Development

No branches or pull requests

2 participants