Skip to content
Permalink
Browse files
Build fix: Don't use __builtin_mul_overflow on PLATFORM(IOS_FAMILY_SI…
…MULATOR).

https://bugs.webkit.org/show_bug.cgi?id=240937

Reviewed by Saam Barati and Alex Christensen.

The __mulodi4 symbol used by clang for __builtin_mul_overflow is not available.

* Source/WTF/wtf/CheckedArithmetic.h:

Canonical link: https://commits.webkit.org/250993@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@294861 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
Mark Lam committed May 26, 2022
1 parent 3ba1f41 commit 2c26e24c2d9dcd855b46523d51a877f8f19eb981
Showing 1 changed file with 2 additions and 2 deletions.
@@ -1,5 +1,5 @@
/*
* Copyright (C) 2011-2021 Apple Inc. All rights reserved.
* Copyright (C) 2011-2022 Apple Inc. All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions
@@ -34,7 +34,7 @@
/* On Linux with clang, libgcc is usually used instead of compiler-rt, and it does
* not provide the __mulodi4 symbol used by clang for __builtin_mul_overflow
*/
#if COMPILER(GCC) || (COMPILER(CLANG) && !(CPU(ARM) && OS(LINUX)))
#if COMPILER(GCC) || (COMPILER(CLANG) && !(CPU(ARM) && OS(LINUX))) && !PLATFORM(IOS_FAMILY_SIMULATOR)
#define USE_MUL_OVERFLOW 1
#endif

0 comments on commit 2c26e24

Please sign in to comment.