[Buildroot] [PATCH/next v2 1/2] valgrind: let the valgrind configure script detect TLS availability

From
Thomas Petazzoni <hidden>
Date
2016-08-31 19:46:18
Reply to
Thomas Petazzoni, 2016-08-30
Hello,

On Tue, 30 Aug 2016 23:33:27 +0200, Thomas Petazzoni wrote:
Back in 2005, in commit
a2c326396a43ecbc8d02c3d815d4010a7ba2e004 ("update valgrind to the latest
and greatest"), an explicit --disable-tls option was added. More
recently, in commit 31a3f4bd54e12b8d6de286ab8fb6d9651990e2f5 ("valgrind:
enable tls support") changed this to be conditional on
BR2_GCC_ENABLE_TLS.

However, the configure script of valgrind is perfectly capable of
detecting TLS support, even in a cross-compilation case: it tries to
compile a program that uses __thread and sees if it works.

Since we're about to modify how BR2_GCC_ENABLE_TLS is handled, we'd
better remove its usage from packages, and valgrind is the only package
using this config option.

Signed-off-by: Thomas Petazzoni <redacted>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <redacted>
---
Changes since v1:
 - Added Arnout Reviewed-by.
---
 package/valgrind/valgrind.mk | 6 ------
 1 file changed, 6 deletions(-)
I've applied both patches to next.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
Thread (5 messages) 5 messages, 2 authors, last reply 2016-08-31
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help