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 flag check for fuzzers #7542

Merged
merged 1 commit into from
May 1, 2023

Conversation

nathaniel-brough
Copy link
Contributor

@nathaniel-brough nathaniel-brough commented May 1, 2023

On some system size_t isn't available under <cstdint>, however it is garaunteed to be available under <cstddef> for all systems.

On some system size_t isn't available under <cstdint>, however
it is garaunteed to be available under <cstddef> for all systems.
@nathaniel-brough
Copy link
Contributor Author

This fixes some build in google/oss-fuzz#10107 failures caused by different implementations of the standard c++ library. Not sure how this one slipped passed me, it was working fine on my local machine and was working fine at some point on the oss-fuzz docker containers as well. Essentially the check to see if fuzzing was supported was failing due to an unrelated reason, (size_t not defined).

@nathaniel-brough
Copy link
Contributor Author

So that definetely fixes the cmake flag check issue https://github.com/silvergasp/oss-fuzz/actions/runs/4854664389/jobs/8652309043#step:8:8960 but now I'm running into some other funky runtime issue, when actually running the fuzzers on the oss-fuzz build...

@steven-johnson steven-johnson merged commit 6db47d3 into halide:main May 1, 2023
ardier pushed a commit to ardier/Halide-mutation that referenced this pull request Mar 3, 2024
On some system size_t isn't available under <cstdint>, however
it is garaunteed to be available under <cstddef> for all systems.
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