Bug 242554 – Shall the workaround for llvm.37598 be removed?

• Status: NEW
• Component: WebRTC

I find a workaround for llvm.37598 in line 86 of mem.c:
https://github.com/WebKit/WebKit/blob/main/Source/ThirdParty/libwebrtc/Source/third_party/boringssl/src/crypto/mem.c

// Windows doesn't really support weak symbols as of May 2019, and Clang on
// Windows will emit strong symbols instead. See
// https://bugs.llvm.org/show_bug.cgi?id=37598
#if defined(__ELF__) && defined(__GNUC__)
#define WEAK_SYMBOL_FUNC(rettype, name, args) \
  rettype name args __attribute__((weak));
#else
#define WEAK_SYMBOL_FUNC(rettype, name, args) static rettype(*name) args = NULL;
#endif

This compiler bug is already fixed: https://bugs.llvm.org/show_bug.cgi?id=37598
Shall these workarounds be removed?

[Radar WebKit Bug Importer]

<rdar://problem/97121791>