Skip to content

Fix regular expression aggressive elim#6627

Merged
4tXJ7f merged 5 commits intocvc5:masterfrom
ajreynol:fixReAgg
May 27, 2021
Merged

Fix regular expression aggressive elim#6627
4tXJ7f merged 5 commits intocvc5:masterfrom
ajreynol:fixReAgg

Conversation

@ajreynol
Copy link
Copy Markdown
Member

@ajreynol ajreynol commented May 27, 2021

Fixes #6620, fixes #6622. Fixes cvc5/cvc5-projects#254.

The benchmarks from the 2 issues timeout, a regression is added for the projects issue.

@ajreynol ajreynol added major Priority simple Complexity labels May 27, 2021
@ajreynol ajreynol requested a review from 4tXJ7f May 27, 2021 17:43
Copy link
Copy Markdown
Member

@4tXJ7f 4tXJ7f left a comment

Choose a reason for hiding this comment

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

LGTM

{
sStartIndex = lens;
}
else if (r==1 && sConstraints.size()==2)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Apply ClangFormat.

@4tXJ7f 4tXJ7f added the do not merge Do not merge this PR if you are not the author label May 27, 2021
@ajreynol ajreynol removed the do not merge Do not merge this PR if you are not the author label May 27, 2021
@ajreynol ajreynol changed the title Fix regular expression aggressive rewrite Fix regular expression aggressive elim May 27, 2021
@4tXJ7f 4tXJ7f enabled auto-merge (squash) May 27, 2021 23:22
@4tXJ7f 4tXJ7f merged commit a1624b7 into cvc5:master May 27, 2021
@ajreynol ajreynol deleted the fixReAgg branch May 28, 2021 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major Priority simple Complexity

Projects

None yet

2 participants