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

fix rlimit for clang-10 #4658

Merged
merged 1 commit into from
Aug 21, 2020
Merged

Conversation

agurfinkel
Copy link
Collaborator

Alternative fix to reslimit::push(0)

@NikolajBjorner NikolajBjorner merged commit 22b5daf into Z3Prover:master Aug 21, 2020
@NikolajBjorner
Copy link
Contributor

Did you try

uint64_t new_limit = m_count + static_cast<uint64_t>(delta_limit);

@nunoplopes - is the compiler right?

@agurfinkel
Copy link
Collaborator Author

@NikolajBjorner your suggestion does not work. Same issue as before.

@agurfinkel agurfinkel deleted the rlimit_fix2 branch August 21, 2020 17:57
NikolajBjorner added a commit that referenced this pull request Aug 22, 2020
* extended calculation of info for regexes, updated tracing of state_graph with regex info

* took care of PR comments and fixed some info calculation bugs

* fix rlimit for clang-10 (#4658)

* merge

Signed-off-by: Nikolaj Bjorner <nbjorner@microsoft.com>

* extended calculation of info for regexes, updated tracing of state_graph with regex info

* took care of PR comments and fixed some info calculation bugs

* added missing return statements, reordered def of compl to match declaration order of methods

* fixed loop lower bound bug in loop info and default nullable value in invalid_info

* fixed type bug: bool to lbool

* trying to remove invisible control characters

* renamed compl method (compl is a reserved c++ keyword) to complement

Co-authored-by: Arie Gurfinkel <arie.gurfinkel@uwaterloo.ca>
Co-authored-by: Nikolaj Bjorner <nbjorner@microsoft.com>
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.

None yet

2 participants