Skip to content

Add project to fuzz PyPy3#15355

Open
StanFromIreland wants to merge 43 commits into
google:masterfrom
StanFromIreland:pypy3
Open

Add project to fuzz PyPy3#15355
StanFromIreland wants to merge 43 commits into
google:masterfrom
StanFromIreland:pypy3

Conversation

@StanFromIreland
Copy link
Copy Markdown
Contributor

@StanFromIreland StanFromIreland commented Apr 13, 2026

I had to do some funky things in the build script to get instrumentation working, but it seems to build now.

Currently, the targets live in https://github.com/StanFromIreland/pypy-fuzz.git, but this may change in the future.

CC @cfbolz and @mattip

@github-actions
Copy link
Copy Markdown

StanFromIreland is integrating a new project:
- Main repo: https://github.com/pypy/pypy
- Criticality score: 0.58880

@StanFromIreland StanFromIreland marked this pull request as ready for review April 13, 2026 21:23
@mattip
Copy link
Copy Markdown

mattip commented Apr 14, 2026

Nice. Sorry about the incomplete Makefile, fixes welcome. It might need a make install command?

It seems the fuzzing is passing? Or is it not running yet?

@StanFromIreland
Copy link
Copy Markdown
Contributor Author

StanFromIreland commented Apr 14, 2026

It seems the fuzzing is passing? Or is it not running yet?

The fuzzer builds, it just isn't running yet (as the PR hasn't been merged).

Comment thread projects/pypy3/project.yaml Outdated
Comment thread projects/pypy3/project.yaml
Comment thread projects/pypy3/build.sh
@StanFromIreland
Copy link
Copy Markdown
Contributor Author

StanFromIreland commented Apr 14, 2026

@DavidKorczynski, builds with ASAN currently fail. I tried some things, but I couldn't fix it. Is it possible to just have UBSan for now, and in the future when we figure out what's going on with ASAN add it in another PR? Furthermore, becasue of the GC, ASAN isn't a priority.

@StanFromIreland
Copy link
Copy Markdown
Contributor Author

@DavidKorczynski Friendly ping :-)

Copy link
Copy Markdown
Collaborator

@DavidKorczynski DavidKorczynski left a comment

Choose a reason for hiding this comment

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

Overall this looks good -- but the run_tests.sh is just an empty file? No need to include it in case.

@StanFromIreland
Copy link
Copy Markdown
Contributor Author

Thanks for the review, I've removed it.

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.

3 participants