Same wife/person. Different name, email.
-benjamin
2013-08-19 Benjamin De Kosnik b...@gnu.org
* MAINTAINERS: Update name, email.
diff --git a/MAINTAINERS b/MAINTAINERS
index ef6ab87..c0ffb68 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -194,7 +194,7 @@ dwarf debugging code Cary Coutant
More patches to fix disable issus on non-linux.
Here's a patch for the build failure on darwin.
Will check in when testing completes.
tested x86/linux
tested x86_64/darwin12
-benjamin2013-08-08 Benjamin Kosnik b...@rehat.com
Michael Meissner meiss...@linux.vnet.ibm.com
*
Here's a patch for the build failure on darwin.
tested x86/linux
-benjamindiff --git a/libvtv/Makefile.am b/libvtv/Makefile.am
index ab3233f..73acfb4 100644
--- a/libvtv/Makefile.am
+++ b/libvtv/Makefile.am
@@ -19,7 +19,11 @@
## along with this library; see the file COPYING3. If not see
##
+# Filter out unsupported systems.
+case ${target} in
+ x86_64-*-linux* | i?86-*-linux*)
+ VTV_SUPPORTED=yes
+ ;;
+ powerpc*-*-linux*)
+ ;;
+ sparc*-*-linux*)
+ ;;
+ arm*-*-linux*)
+ ;;
What about powerpc, sparc and arm? Why are they mentioned here if no
actual
Nice to see!
I have created (with some help) a git branch on gcc.gnu.org to contain
the vtable verification feature work. This work is now well
integrated with GCC trunk, and the sources are in a good state for
future work. I believe all previous review comments have been
addressed.
The
Some of the DR130-mandated signature changes for C++11 require the use
of abi_tag to properly distinguish from C++98 calls of the same
function, but with a different return type.
I've audited libstdc++ for C++98/C++11 API differences that are
signature changes that mangle the same. C++11 to
Noticed this typo when doing the 4.8.1 docs. I'd removed this in the
top-level file before, and that's fine, but these release-versioned
files were wrong.
-benjamin2013-05-31 Benjamin Kosnik b...@redhat.com
* htdocs/onlinedocs/4.8.0/index.html: Correct to GFDL.
*
It scraps the renaming/aliasing approach, and just creates a
compatibility-chrono.cc that mimics the default configuration in
4.8.0.
Yeah, I think that is reasonable, with one nit, see below.
Cool, incorporated.
Users who specially-configured a build with --enable-libstdcxx-time
Looks good to me.
Great, trunk patch in.
Will you prepare corresponding patch for the 4.8
branch too (I guess it should be pretty much the same, not sure if it
should keep using typedef system_clock steady_clock as before for the
non-MONOTONIC configuration, or if it should just do the
I get the
following testsuite regressions on Solaris 10/x86:
FAIL: 30_threads/async/54297.cc (test for excess errors)
WARNING: 30_threads/async/54297.cc compilation failed to produce
executable FAIL: 30_threads/condition_variable_any/53830.cc (test
for excess errors) WARNING:
So, there is a minor issue that what is std::chrono::steady_clock has
changed, if you say use it as a function parameter, it will mangle
differently before/after. Guess not that big a deal, after all, C++11
support is still experimental, right?
Right, ditto, yes.
But the more important
Fix typo in assignment email address, to make gcc match GNU site:
https://www.gnu.org/prep/maintain/maintain.html#Legal-Matters
It would be awesome if now GCC assigments were magically less work.
-benjamin2013-05-22 Benjamin Kosnik b...@redhat.com
* htdocs/contribute.html: Use
I'd like to know if someone with better automake skills than I have
can do anything about that warning, but otherwise that looks OK to me,
thanks.
Hey dude. Caroline and I are working off-line on this.
-benjamin
Some cleanup, no Makefile.am was checked in on the bad_array_*
additions. The exports make the intent very clear, so this just fills
in the blanks.
tested x86/linux
-benjamin2013-05-13 Benjamin Kosnik b...@redhat.com
* libsupc++/Makefile.am (sources): Add bad_array_length.cc,
This updates the links for the 4.7.3 and 4.6.4 libstdc++ documentation.
Now 4.6-4.8 links are all consistent.
best,
-benjamin
2013-04-22 Benjamin Kosnik b...@redhat.com
* htdocs/onlinedocs/index.html: Adjust GDL to GFDL. Change
4.7.3 and 4.6.4 to use gz instead of bz2
Slight update to links, reflecting more attempts by libstdc++ to blend
in with the native layout and compression choices for gcc releases.
Links check so seems fine.
While at this, I figured I might as well do the GDL/GPL dupe for the
libstdc++-api XML file, as discussed at Cauldron last year.
Hey! Here is the first pass at the 4.8 porting documentation.
This seems to reflect the current trunk reality. I'm not quite sure to
about the best way to talk about the more aggressive loop
optimizations WRT undefined sematincs, but this seems reasonable. Of
course, if anybody has better
It doesn't make sense. memcpy from NULL src pointer?
Indeed, that doesn't make sense. Thanks.
-benjamin
Here is round two, as checked-in.
-benjamin
2013-03-13 Benjamin Kosnik b...@redhat.com
* htdocs/gcc-4.8/porting_to.html: Add.
* htdocs/gcc-4.8/changes.html: Add link.
Index: changes.html
===
RCS file:
Here is round two, as checked-in.
... and here are the validation patches.
-benjamin2013-03-13 Benjamin Kosnik b...@redhat.com
* htdocs/gcc-4.8/porting_to.html: Fix markup.
* htdocs/gcc-4.8/changes.html: Same.
Index: changes.html
Sync html in sources with generated output.
-benjamin
20130313-7-docs.patch.bz2
Description: application/bzip
Hey y'all.
I noticed the repeated, repeated, repeated use of
std::allocator forward decls as I was trying to debug some doxygen
output.
Clearly, it's past time for a memory forward header, like
bits/stringfwd.h. The following patch implements this. Perhaps some of
the other headers can now be
How about the attached file as a start for ext/math. I used the
constexpr approach (instead of function calls) and replicated the
constants that are available in math.h in Unix.
then this should really be
ext/cmath
1) In this case I miss the corresponding variable definitions,
Fixups for doxygen. Since 2012-11-20, most of C++11 has been ignored by
doxygen, resulting in some missing modules documentation even in HTML
output.
Jakub, this fixes the libstdc++.tag validation errors.
I've regenerated the gcc.gnu.org docs for 4.8.0. They look equivalent or
better than 4.7.2
Does this configury/feature macro handling look right?
yes.
Is this enough testcase?
yes.
Is this small/safe enough to go in for 4.8?
yes.
commit 884a2a7815a95bade9d23f01b4c64a16808c7f05
Author: Jason Merrill ja...@redhat.com
Date: Mon Feb 11 11:23:30 2013 -0500
*
.. also, I think we should have #if __cplusplus = 201103L protecting
these functions too, exactly like all the other C++11 bits. Eg, I
don't think we should unconditionally, ie in C++03 mode too, declare
at_quick_exit and quick_exit in namespace std.
Yeah.
I can tweak things for the two
Since commit r195676[1], it looks like
libstdc++-v3/src/c++11/hashtable_c++0x.cc is missing an explicit
instantiation for std::lower_bound.
it's missing an implicit instantiation of std::lower_bound.
This leads to
Fixup for a typo, on trunk and 4.7
-benjamin2012-12-12 Benjamin Kosnik b...@redhat.com
* doc/xml/manual/documentation_hacking.xml: Fix validation issue.
diff --git a/libstdc++-v3/doc/xml/manual/documentation_hacking.xml b/libstdc++-v3/doc/xml/manual/documentation_hacking.xml
index
libstdc++-v3/doc/xsl/customization.xsl.in is marked as
svn:mime-type = application/xml
at least on the 4.7 branch, having some unexpected outcome for svn
diff. If this was unintended, could you change the svn:mime-type
back to text?
This is unintentional, and frankly I don't
Missing exports for unordered_* containers in debug mode.
The patches between mainline/4.7 are a slight bit different, but
equivalent.
tested x86/linux
tested x86/linux --enable-symvers=gnu-versioned-namespace
2012-11-05 Benjamin Kosnik b...@redhat.com
Oleg Smolsky
as per LWG comments on N330[2-5].
Couple of notes: new fail has been temporarily worked around pending
the resolution of
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54968
For tuple, the big fun remaining is piecewise_construct_t, which
should allow tie and forward_as_tuple to be
I don't like the sched_yield macro being set there because it's
detected correctly by configure anyway, but I'm not going to labour
that point any more.
Indeed. Then somebody will waste hours in the future wondering why
configure says no but their TU says yes.
At least a comment in the
Small fix for 32 bit targets.
-benjamin2012-10-09 Benjamin Kosnik b...@redhat.com
* testsuite/20_util/specialized_algorithms/uninitialized_copy/808590.cc:
Fix constant value.
diff --git a/libstdc++-v3/testsuite/20_util/specialized_algorithms/uninitialized_copy/808590.cc
2012-10-09 Uros Bizjak ubiz...@gmail.com
* testsuite/util/testsuite_abi.cc (check_version): Add
CXXABI_1.3.7.
Tested on x86_64-pc-linux-gnu.
OK for mainline?
Ok, thanks!
-benjamin
On Android NDK libstdc++ is configured, built and packaged separately.
The problem is not dependency on libgcc sources but rather dependency
on the symlink which is generated during libgcc build and cannot be
found if libstdc++ is configured and built separately.
It was working fine for 4.4
More testsuite changes, tracking GSOC work.
tested x86/linux
-benjamin2012-10-05 Benjamin Kosnik b...@redhat.com
* testsuite/28_regex/algorithms/match: Rename to...
* testsuite/28_regex/algorithms/regex_match: ...this.
* testsuite/28_regex/basic_regex/regex.cc: Rename to...
*
to 1.8.2, yay, fixes for template aliases in!
tested x86/linux
-benjamindiff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog
index 9302b71..33a46cc 100644
--- a/libstdc++-v3/ChangeLog
+++ b/libstdc++-v3/ChangeLog
@@ -1,3 +1,7 @@
+2012-09-30 Benjamin Kosnik b...@redhat.com
+
+ *
adds in exports for more vtable bits, as per bugzilla
missed some, added here and tested with:
tested x86/linux --enable-symvers=gnu-versioned-namespace
-benjamin2012-10-01 Benjamin Kosnik b...@redhat.com
* config/abi/pre/gnu-versioned-namespace.ver: Add more
typeinfo/vtable exports.
Hey Iain.
I looked at this, and easily reproduced your results.
The following symbols are actually required by all
namespace associated-modes, not just parallel, and do not require
parallel capability:
Symbols reported as missing by the abi.exp code:
... found while working on arm-eabisim cross, using --enable-cxx-flags
was not working as AM_CXXFLAGS was being over-ridden by CXXFLAGS.
(Despite the comments warning about this.)
Fixed.
Also patchlet for the last commit, forgot to edit PARALELL_FLAGS, so
--disable-thread compiles were failing.
Fixes for various shared/static/pic issues, see SUBJECT.
tested x86_64/linux
tested x86_64/linux --with-pic
tested x86_64/linux --without-pic*
tested x86_64/linux --disable-shared
tested x86_64/linux --disable-static
earlier versions tested on cross arm-eabisim
Most of these do what you'd
On Thu, 30 Aug 2012 12:48:34 +0200
Thiago Macieira thiago.macie...@intel.com wrote:
Hello
The attached patch is a simple improvement to make a thread that
failed to set the waiting bit to exit the function earlier, if it
detects that another thread has successfully finished initialising.
On Wed, 29 Aug 2012 14:34:40 -0400
Ulrich Drepper drep...@gmail.com wrote:
On Wed, Aug 29, 2012 at 11:43 AM, Paolo Carlini
paolo.carl...@oracle.com wro
The substance isn't of course. But normally we don't have __gnu_cxx
things in the same std header. Can't we have a new ext/random and
put
Pretty minor change, as per PR. This version seems more appropriate
for templatized types.
.. this wasn't right, commentary as per bugzilla.
tested x86/linux
-benjamin
2012-08-07 Benjamin Kosnik b...@redhat.com
PR libstdc++/54005 continued
* include/std/atomic: Use __atomic_lock_free
... finally got around to fixing up the generated HTML, so that charset
is set to UTF-8.
Sorry this took so long.
tested x86/linux
-benjamindiff --git a/libstdc++-v3/configure.ac b/libstdc++-v3/configure.ac
index 3853472..5c86bb9 100644
--- a/libstdc++-v3/configure.ac
+++
Pretty minor change, as per PR. This version seems more appropriate for
templatized types.
I'll wait a bit before putting this on 4.7_branch
-benjamin
tested x86/linux
2012-08-07 Benjamin Kosnik b...@redhat.com
PR libstdc++/54005
* include/std/atomic: Use
Removes duplicate tests file, makes sub-directory naming consistent
tested x86/linux
-benjamin2012-06-24 Benjamin Kosnik b...@redhat.com
* testsuite/28_regex/algorithms/02_match: To...
* testsuite/28_regex/algorithms/match: ...this.
*
Here's a deceptively small patchlet that allows make doc-pdf-doxygen
build the api PDF file when the prerequisites are met. (And the
texmf.cnf file is edited to increase the underlying TeX
subsystem's memory.) I'm currently using doxygen 1.8.1.1 and a variety
of latex subsystems, depending on the
Renames to avoid builtins on clang.
(This patch of code is actually the punchline to the joke, how many
template aliases are really required if you are going for no line
breaks and using 80 columns while using policy based designs.)
tested x86/linux
-benjamin2012-05-31 Benjamin Kosnik
As per libstdc++/52700, this fixes the configure bits for
libgfortran/libssp/libquadmath. With these fixes, I believe all the
libs are safe for --enable-symvers=gnu* variants.
Super simple patches...
I intend to put this on the 4.7 branch as well.
tested x86/linux
-benjamin
2012-05-29
Allows use of --enable-symvers=gnu-versioned-namespace while
configuring in libjava. The rest of the target libs that use
--enable-symvers already handle this. As per libstdc++/52700.
Pretty simple, but will wait for OK for trunk/branch
tested x86/linux
trunk
4_7-branch
-benjamin2012-05-16
In libstdc++/44015, there is a request for doxygen markup on template
parameters, including default arguments. This is a kind of markup that
libstdc++ has mostly not done, or done inconsistently.
So, here's how I think it should be done, at least for io and
containers.
See generated files
Enables -ftrack-macro-expansion when running libstdc++ testsuite.
tested x86/linux
-benjamin2012-05-03 Benjamin Kosnik b...@redhat.com
* scripts/testsuite_flags.in (cxxflags): Remove
-ftrack-macro-expansion=0.
diff --git a/libstdc++-v3/scripts/testsuite_flags.in
* crossconfig.m4: Since we know that all TPF builds are cross-
builds and cannot run configuration-time link tests, do not
allow it; just go with known supported linker options.
* configure: Regenerate (called as GLIBCXX_CROSSCONFIG).
OK
-benjamin
This patchlette needed for versioned builds, just makes
declaration/foward declaration consistent.
-benjamin
tested x86/linux
tested x86/linux versioned namespacesdiff --git a/libstdc++-v3/include/bits/hashtable_policy.h b/libstdc++-v3/include/bits/hashtable_policy.h
index 3b43510..a42d3d3
... putting in line breaks for doxygen, so that each base class is on a
separate line.
-benjamin2012-04-25 Benjamin Kosnik b...@redhat.com
* include/bits/hashtable.h: Adjust doxygen markup for base classes.
* include/bits/hashtable_policy.h: Same.
diff --git
Noticed that this testcase wasn't put in as part of the patch. Fixed as
follows.
tested x86/linux
-benjamin2012-04-24 Benjamin Kosnik b...@redhat.com
PR libstdc++/52689
* testsuite/17_intro/static.cc: New.
diff --git a/libstdc++-v3/testsuite/17_intro/static.cc
Found this bug in 4.4 branches, fixed with the following in later
branches:
http://gcc.gnu.org/ml/gcc-patches/2010-06/msg01616.html
But the test case is useful anyway.
tested x86/linux
-benjamin2012-04-16 Benjamin Kosnik b...@redhat.com
*
This patch restructures the std::unordered_* containers, as per the
tempting comment in unordered_set:
// NB: When we get typedef templates these class definitions
// will be unnecessary.
This required some changes in _Hashtable, forced the consolidation of
insert() logic in a base class,
Seems like libstdc++.a has suffered a bit in the transition to a more
modular convenience-library build system.
Here's a patch to clean up the compatibility symbols throughout
libstdc++, to insure that necessary symbols for versioning purposes are
only in shared binaries, ie libstdc++.so. Most
Removes duplicates on command line when running test suite. CXXFLAG
info was being quoted twice.
tested x86/linux
-benjamin
... here's the patch.2012-03-22 Benjamin Kosnik b...@redhat.com
* scripts/testsuite_flags.in (--cxxflags): Let CXXFLAGS set -g -O2.
* testsuite/lib/libstdc++.exp: Same.
diff --git a/libstdc++-v3/scripts/testsuite_flags.in b/libstdc++-v3/scripts/testsuite_flags.in
index 457adaf..f77784b
Seems like -fpph doesn't work on the pph branch anymore, and instead
-fpph-gen/-fpph-map/-fpph-hdr are used.
The thing is, how are these flags used?
unhelpful wiki page:
http://gcc.gnu.org/wiki/pph
Based on invoke.texi, I'm expecting something like this:
1. generate
g++ -fpph-gen all.h;
//
Small filename tweaks to gcc's docs page for the generated 4.7.0
libstdc++ docs.
-benjamin2012-03-22 Benjamin Kosnik b...@redhat.com
* htdocs/onlinedocs/index.html: Adjust libstdc++ xml file names.
Index: htdocs/onlinedocs/index.html
Simple rename for XML filenames. The -single can be omitted.
tested x86/linux
-benjamin2012-03-22 Benjamin Kosnik b...@redhat.com
* doc/Makefile.am: XML output names have only .xml suffix.
* doc/Makefile.in: Regenerate.
diff --git a/libstdc++-v3/doc/Makefile.am
this removes the pb_ds fails exposed by this patch.
-benjamin2012-03-19 Benjamin Kosnik b...@redhat.com
* include/ext/pb_ds/detail/pat_trie_/
constructors_destructor_fn_imps.hpp: Increment after recursion.
* include/ext/pb_ds/detail/pat_trie_/pat_trie_base.hpp: Convert
On Fri, 16 Mar 2012 11:10:48 +0100
Richard Guenther richard.guent...@gmail.com wrote:
On Fri, Mar 16, 2012 at 1:29 AM, Jonathan Wakely
jwakely@gmail.com wrote:
On 15 March 2012 15:40, Richard Guenther wrote:
On Thu, Mar 15, 2012 at 4:22 PM, Kai Tietz
ktiet...@googlemail.com wrote:
... removes extra extern.
-benjamin2012-03-02 Benjamin Kosnik b...@redhat.com
* include/c_global/cstdio: Remove extraneous extern.
* include/c_std/cstdio: Same.
diff --git a/libstdc++-v3/include/c_global/cstdio b/libstdc++-v3/include/c_global/cstdio
index d11743a..e648475 100644
I was surprised to see GLIBCXX_3.4.15 symbols added, but then realized
you added the complete set so this seems fine.
I meant to say: surprised to see GLIBCXX_3.4.16 symbols added, but then
you the complete set so this seems fine. So the only added symbols are
the complete set of
noticed these bits hanging out after a make clean in testsuite. Fixed.
tested x86/linux
-benjamin2012-03-01 Benjamin Kosnik b...@redhat.com
* testsuite/Makefile.am (CLEANFILES): Add *.gdb.
* testsuite/Makefile.in: Regenerate.
diff --git a/libstdc++-v3/testsuite/Makefile.am
After discussions in Kona, some suggestions to refine this trait were
proposed, as attached. In addition, there is some documentation which
will be checked in later.
tested x86/linux
-benjamin2012-03-01 Michael Spertus mike_sper...@symantec.com
* include/tr2/type_traits (typelist): To
as per the bugzilla log
tested x86/linux
-benjamin2012-03-01 Benjamin Kosnik b...@redhat.com
Ramana Radhakrishnan ram...@gcc.gnu.org
PR libstdc++/51785
* acinclude.m4 (GLIBCXX_CHECK_STDIO_PROTO): New.
* configure.ac: Call it.
* configure: Regenerate.
* config.h.in: Same.
*
as requested. all new symbols should be in new symbol versoning names.
With this in, check-abi will now fail if new symbols are inadvertently
added to previously-released versions.
I'm expecting this to make solaris symbol versioning, as reported by
check-abi, fail at first. Then we can
Debugging pdf output on current linux indicates that the long-standing
PDF_HYPERLINKS issue turns out to just be a pool_size limitation in
TeX, which can be worked around by increasing pool size. Document.
tested x86_64/linux
-benjamin
2012-02-07 Benjamin Kosnik b...@redhat.com
*
74 matches
Mail list logo