This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] fix #19444 - build failures with -O1 due to -Wmaybe-uninitialized
- From: Joseph Myers <joseph at codesourcery dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Martin Sebor <msebor at gmail dot com>, GNU C Library <libc-alpha at sourceware dot org>
- Date: Thu, 14 Jan 2016 18:06:49 +0000
- Subject: Re: [PATCH] fix #19444 - build failures with -O1 due to -Wmaybe-uninitialized
- Authentication-results: sourceware.org; auth=none
- References: <56968319 dot 3070802 at gmail dot com> <alpine dot DEB dot 2 dot 10 dot 1601131740310 dot 17296 at digraph dot polyomino dot org dot uk> <CAMe9rOpBfq9q2X3+7GKe4APKuR6zPnzxDqsv86_aLAw+uN+vhg at mail dot gmail dot com> <alpine dot DEB dot 2 dot 10 dot 1601131756510 dot 17296 at digraph dot polyomino dot org dot uk> <CAMe9rOoHDQCOjs9HT-6fo5ayrLd-4d7VuyZa9xJMJkza9kE5Jg at mail dot gmail dot com>
On Wed, 13 Jan 2016, H.J. Lu wrote:
> > It would be good to have an updated list of issues seen. I'd expect most
> > such issues to be easy to fix (adding missing *_hidden_* for functions
> > where PLT avoidance currently relies on inlining, for example).
>
> This is a regression:
>
> https://sourceware.org/bugzilla/show_bug.cgi?id=19462
I think the right way to fix those _STRING_ARCH_unaligned issues is: get
Wilco's patch <https://sourceware.org/ml/libc-alpha/2016-01/msg00170.html>
reviewed (which we need to do for 2.23 anyway as an ABI issue for
AArch64), then move _STRING_ARCH_unaligned to a separate non-installed
header, not in bits/, which is included directly by the files that need
it. (It's possible that Wilco's patch depends on other patches to
eliminate _STRING_ARCH_unaligned tests from installed headers except where
they are replaced by _STRING_INLINE_unaligned.)
--
Joseph S. Myers
joseph@codesourcery.com