beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-25 09:00:22 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

AK+Meta: Disable consteval completely when building for oss-fuzz

Browse source 
This was missed in 02b74e5a70

We need to disable consteval in AK::String as well as AK::StringView,
and we need to disable it when building both the tools build and the
fuzzer build.
This commit is contained in:
Andrew Kaster 2023-06-29 14:35:03 -06:00 committed by Andrew Kaster
parent 5bc2c333bd
commit bfd6deed1e
Notes: sideshowbarker 2024-07-16 23:34:49 +09:00 
Author: https://github.com/ADKaster
Commit: https://github.com/SerenityOS/serenity/commit/bfd6deed1e
Pull-request: https://github.com/SerenityOS/serenity/pull/19711
2 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
AK/String.h
View file

@ -30,7 +30,8 @@ class StringData;
}
// FIXME: Remove this when OpenBSD Clang fully supports consteval.
#if defined(AK_OS_OPENBSD)
// And once oss-fuzz updates to clang >15.
#if defined(AK_OS_OPENBSD) || defined(OSS_FUZZ)
# define AK_SHORT_STRING_CONSTEVAL constexpr
#else
# define AK_SHORT_STRING_CONSTEVAL consteval

5
Meta/Lagom/BuildFuzzers.sh
View file

@ -46,10 +46,15 @@ unset CFLAGS
unset CXXFLAGS
export AFL_NOOPT=1
if [ "$#" -gt "0" ] && [ "--oss-fuzz" = "$1" ] ; then
CXXFLAGS="$CXXFLAGS -DOSS_FUZZ=ON"
fi
# FIXME: Replace these CMake invocations with a CMake superbuild?
echo "Building Lagom Tools..."
cmake -GNinja -B Build/tools \
-DBUILD_LAGOM=OFF \
-DCMAKE_CXX_FLAGS="$CXXFLAGS" \
-DCMAKE_INSTALL_PREFIX=Build/tool-install \
-Dpackage=LagomTools
ninja -C Build/tools install