This is the mail archive of the mailing list for the elfutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] configure: add a --disable-werror option

On Thu, 2014-01-02 at 17:48 -0500, Mike Frysinger wrote:
> the majority of those changes are just for old toolchains (looks like gcc-4.3 
> is the oldest version that'll work, but that's what we've been requiring in 
> glibc for a few releases and no one seems to mind, so shouldn't be an issue).
> i think these belong in the main branch though:
> 35ae9cf964c2de52bb4d03412334fe83338393ee
> - adds an abort() after an assert()
> if you build with -DNDEBUG, the assert() is a NOP, and the code can continue 
> running when it clearly shouldn't.

More stuff breaks when you build with -DNDEBUG. Are you sure we want to
support that setup? In general we seem to use assert and/or abort a lot
for non-reachable code paths. It would be nice to have a better solution
for that, especially in places where the compiler would be able to check
that assertion. I am not sure what the correct way to express that is

> a3462c319770a035e8a1deed5b49f5a0d95c5a2b
> - support systems that lack tls
> probably not terribly important as every arch that is supported by current 
> glibc has tls support now, but not every C library supports tls.  plus, this 
> change has like no overhead for the project (it's a minor tweak to 
> only).

That is for the unsupported configure option --enable-thread-safety for
which we already warn it should not be used.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]