From 02b74e5a70e1eef2fe3123fe8aa157301424783a Mon Sep 17 00:00:00 2001 From: Andrew Kaster Date: Tue, 27 Jun 2023 19:31:58 -0600 Subject: [PATCH] AK: Re-enable consteval StringView literal workaround for oss-fuzz oss-fuzz ships a pre-release commit of clang-15 for all of their build bots. Until they update to a release of clang-15 that includes the fix for this bug, or a later release, we need to keep the workaround in place. --- AK/StringView.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/AK/StringView.h b/AK/StringView.h index 77ce92a70b..d86a42b5e6 100644 --- a/AK/StringView.h +++ b/AK/StringView.h @@ -374,7 +374,9 @@ struct CaseInsensitiveASCIIStringViewTraits : public Traits { // FIXME: Remove this when clang on BSD distributions fully support consteval (specifically in the context of default parameter initialization). // Note that this is fixed in clang-15, but is not yet picked up by all downstream distributions. // See: https://github.com/llvm/llvm-project/issues/48230 -#if defined(AK_OS_BSD_GENERIC) +// Additionally, oss-fuzz currently ships an llvm-project commit that is a pre-release of 15.0.0. +// See: https://github.com/google/oss-fuzz/issues/9989 +#if defined(AK_OS_BSD_GENERIC) or defined(OSS_FUZZ) # define AK_STRING_VIEW_LITERAL_CONSTEVAL constexpr #else # define AK_STRING_VIEW_LITERAL_CONSTEVAL consteval