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

WIP: Add new regexp fuzz tests that run with unicode input #5573

Closed
wants to merge 22 commits into from

Conversation

andygrove
Copy link
Contributor

Related issue: #5549

These are new tests that I would like to add but they are failing in CI for some reason and I am trying to debug why.

@andygrove
Copy link
Contributor Author

build

Signed-off-by: Andy Grove <andygrove@nvidia.com>
@andygrove
Copy link
Contributor Author

build

@andygrove
Copy link
Contributor Author

build

@andygrove
Copy link
Contributor Author

andygrove commented May 24, 2022

CI fails with:

javaPattern[1]=\z\u20282, cudfPattern=$\u20282, input='????', cpu=????, gpu=???????????? (RegularExpressionTranspilerSuite.scala:728)

Running the same test locally, it passes with this output:

javaPattern[1]=\z\u20282, cudfPattern=$\u20282, input='鈻瑜㶯眀', cpu=鈻瑜㶯眀, gpu=鈻瑜㶯眀

@anthony-chang
Copy link
Contributor

anthony-chang commented May 24, 2022

Also related to issue #5521 (likely the same underlying problem)

@andygrove
Copy link
Contributor Author

build

@andygrove
Copy link
Contributor Author

I will create a new PR since this builds from several in-flight PRs

@andygrove andygrove closed this May 26, 2022
@andygrove andygrove deleted the regexp-fuzz-unicode branch May 17, 2023 17:53
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.

2 participants