From 421ebc2e297e504f07afce76708eb37eb3da9c82 Mon Sep 17 00:00:00 2001 From: Timothy Flynn Date: Tue, 15 Nov 2022 07:12:53 -0500 Subject: [PATCH] LibJS: Remove workaround for Clang misalignment in MathematicalValue As noted in 269a931, this has been fixed in Clang 15. --- Userland/Libraries/LibJS/Runtime/Intl/MathematicalValue.h | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/Userland/Libraries/LibJS/Runtime/Intl/MathematicalValue.h b/Userland/Libraries/LibJS/Runtime/Intl/MathematicalValue.h index 6a8160f6a7..53ff627217 100644 --- a/Userland/Libraries/LibJS/Runtime/Intl/MathematicalValue.h +++ b/Userland/Libraries/LibJS/Runtime/Intl/MathematicalValue.h @@ -95,11 +95,7 @@ private: static ValueType value_from_number(double number); - // NOTE: The specific alignment is to avoid an UBSAN error with Clang i686, due to Clang - // disagreeing with UBSAN on the alignment of doubles. See: - // https://github.com/llvm/llvm-project/issues/54845 - // https://github.com/SerenityOS/serenity/issues/13614 - alignas(8) ValueType m_value { 0.0 }; + ValueType m_value { 0.0 }; }; }