-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
ensure all builds are optimized #421
Comments
libxml is also built w/o optimizations |
afl might be building with -O0 as well, no -O enforced in here - https://github.com/google/oss-fuzz/blob/master/infra/base-images/base-builder/compile_afl#L26. We enforced -O1 everywhere in chromium and it worked better than -O2 (@oliverchang). We can just add it here - https://github.com/google/oss-fuzz/blob/master/infra/base-images/base-clang/Dockerfile#L38 |
-O1 gives better coverage for fuzzing.
When running experiments locally the workflow is to continuously rebuild the static pages while having a http server launched. This is a bit inconvenient. This commit adds an option to the web module such that both a http server is launched and the static generator continues to generate files as long as the http server is launched. --------- Signed-off-by: David Korczynski <david@adalogics.com>
See e.g. https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=678
It's not totally clear if we should enforce -O1 or -O2.
Also, if the project already has O2, do we want to downgrade it to O1?
The text was updated successfully, but these errors were encountered: