vgvassilev wrote:

> This broke the build on FreeBSD where __emutls_get_address is not provided. 
> This seems sufficient to fix the build:
> 
> ```
> --- clang/lib/Interpreter/IncrementalExecutor.cpp.orig
> +++ clang/lib/Interpreter/IncrementalExecutor.cpp
> @@ -76,7 +76,7 @@
>  // and AIX / z/OS (whose runtimes may not provide the symbol). On those hosts
>  // thread_locals instead rely on process-symbol lookup, unchanged from 
> before.
>  #if defined(LLVM_ON_UNIX) && !defined(__EMSCRIPTEN__) && !defined(_AIX) &&   
>   \
> -    !defined(__MVS__)
> +    !defined(__MVS__) && !defined(__FreeBSD__)
>  extern "C" void *__emutls_get_address(void *);
>  static void *getEmuTLSGetAddressPtr() {
>    return reinterpret_cast<void *>(&__emutls_get_address);
> ```
> 
> Whatever tool did the line wrapping of the commit message is appalling. It's 
> basically unreadable. :(

Sorry about that. I noticed the formatting but I thought it was only on my end. 
Do you mind committing that fix of yours?

https://github.com/llvm/llvm-project/pull/209717
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to