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

[compiler-rt] Update libFuzzer build script to use C++17. #89604

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

fcambus
Copy link
Member

@fcambus fcambus commented Apr 22, 2024

libFuzzer uses std::clamp which was introduced in C++17.

libFuzzer uses std::clamp which was introduced in C++17.
@llvmbot
Copy link
Collaborator

llvmbot commented Apr 22, 2024

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Frederic Cambus (fcambus)

Changes

libFuzzer uses std::clamp which was introduced in C++17.


Full diff: https://github.com/llvm/llvm-project/pull/89604.diff

1 Files Affected:

  • (modified) compiler-rt/lib/fuzzer/build.sh (+1-1)
diff --git a/compiler-rt/lib/fuzzer/build.sh b/compiler-rt/lib/fuzzer/build.sh
index f7f329c0d19c76..f58fd9557ce3d5 100755
--- a/compiler-rt/lib/fuzzer/build.sh
+++ b/compiler-rt/lib/fuzzer/build.sh
@@ -2,7 +2,7 @@
 LIBFUZZER_SRC_DIR=$(dirname $0)
 CXX="${CXX:-clang}"
 for f in $LIBFUZZER_SRC_DIR/*.cpp; do
-  $CXX -g -O2 -fno-omit-frame-pointer -std=c++14 $f -c &
+  $CXX -g -O2 -fno-omit-frame-pointer -std=c++17 $f -c &
 done
 wait
 rm -f libFuzzer.a

@fcambus fcambus merged commit 7911615 into llvm:main Apr 24, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants