Lgtm
On Wed, Nov 21, 2018 at 12:10 PM Stella Stamenova via Phabricator <
revi...@reviews.llvm.org> wrote:

> stella.stamenova created this revision.
> stella.stamenova added reviewers: labath, zturner, asmith.
> Herald added subscribers: lldb-commits, jfb.
>
> Right now only some platforms add pthread to the compilation, however, at
> least one of the tests requires the pthread library on Linux as well. Since
> the library is available, this change adds it by default on Linux.
>
>
> Repository:
>   rLLDB LLDB
>
> https://reviews.llvm.org/D54808
>
> Files:
>   lit/helper/toolchain.py
>
>
> Index: lit/helper/toolchain.py
> ===================================================================
> --- lit/helper/toolchain.py
> +++ lit/helper/toolchain.py
> @@ -78,7 +78,7 @@
>              sdk_path = lit.util.to_string(out)
>              lit_config.note('using SDKROOT: %r' % sdk_path)
>              flags = ['-isysroot', sdk_path]
> -    elif platform.system() in ['OpenBSD']:
> +    elif platform.system() in ['OpenBSD', 'Linux']:
>          flags = ['-pthread']
>
>
>
>
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to