2013/2/11 Georg-Johann Lay a...@gjlay.de:
This patch adds optimized support for (short) (un)signed _Sat _Accum
multiplication (HA, UHA, SA, USA).
The 32-bit multiply uses 64 = 32 * 32 widening multiplication. As a spin-off,
the widening mul is available as [u]mulsidi3 default pattern.
Thomas Koenig wrote:
Am 09.02.2013 11:22, schrieb Tobias Burnus:
Why did you put a FIXME there? What's wrong with adding the directory
here?
I think module files are different enough from include files that I
would like to have them in different directories.
I think they are similar enough
Janus Weil wrote:
here is a simple patch for a rejects-valid problem with deferred
type-bound procedures. This is the patch from comment 5 in the PR. A
simpler (but equivalent) patch can be found in comment 4. Both fix the
problem and are free of testsuite regressions.
Is any of the two
This fixes a oversight when introducing anonymous SSA names.
I managed to change a checking piece to a noop. Oops.
I took the opportunity to move the freelist check earlier,
as the type check would trigger for freed SSA names as well.
Bootstrapped and tested on x86_64-unknown-linux-gnu,
On Mon, Feb 11, 2013 at 7:05 PM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
This patch updates GCC 4.8 changes.html to mention
-maddress-mode=[short|long] for x32. OK to install?
H.J.
---
Index: changes.html
===
RCS file:
On Mon, Feb 11, 2013 at 7:12 PM, Iyer, Balaji V balaji.v.i...@intel.com wrote:
Hello Everyone,
Attached, please find a patch that implements Array Notations for C++.
This patch affects the C++ compiler and should build (or be applied) on top
of the Array Notation for C patch (2 of n:
The test fails on x86_64-apple-darwin10 with
FAIL: 18_support/quick_exit/quick_exit.cc (test for excess errors)
Excess errors:
/opt/gcc/work/libstdc++-v3/testsuite/18_support/quick_exit/quick_exit.cc:36:3:
error: 'at_quick_exit' is not a member of 'std'
On 02/12/2013 12:26 PM, domi...@lps.ens.fr wrote:
The test fails on x86_64-apple-darwin10 with
FAIL: 18_support/quick_exit/quick_exit.cc (test for excess errors)
Excess errors:
/opt/gcc/work/libstdc++-v3/testsuite/18_support/quick_exit/quick_exit.cc:36:3:
error: 'at_quick_exit' is not a member
On 02/12/2013 12:32 PM, Paolo Carlini wrote:
On 02/12/2013 12:26 PM, domi...@lps.ens.fr wrote:
The test fails on x86_64-apple-darwin10 with
FAIL: 18_support/quick_exit/quick_exit.cc (test for excess errors)
Excess errors:
IIRC I have already asked a similar question about atan2, erf, ...
which don't appear in the std namespace on darwin.
Dominique
On 02/12/2013 12:40 PM, domi...@lps.ens.fr wrote:
IIRC I have already asked a similar question about atan2, erf, ...
which don't appear in the std namespace on darwin.
You should check whether darwin by chance uses by default c_std instead
of c_global. The former doesn't provide C++11
here is a simple patch for a rejects-valid problem with deferred
type-bound procedures. This is the patch from comment 5 in the PR. A
simpler (but equivalent) patch can be found in comment 4. Both fix the
problem and are free of testsuite regressions.
Is any of the two versions still ok for
The libffi library, headers and documentation are still installed, although
libffi provides separate releases for a long time. So do not install these
anymore as part of a GCC install. Tested with a build and an install with go
and java enabled (both using libffi_convenience). Ok for the trunk?
Test results with this patch:
http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg01351.html
2013-02-12 Sebastian Huber sebastian.hu...@embedded-brains.de
PR target/55033
* gcc.target/powerpc/pr55033.c: New.
---
gcc/testsuite/gcc.target/powerpc/pr55033.c | 21
On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose d...@ubuntu.com wrote:
The libffi library, headers and documentation are still installed, although
libffi provides separate releases for a long time. So do not install these
anymore as part of a GCC install. Tested with a build and an install
On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener
richard.guent...@gmail.com wrote:
On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose d...@ubuntu.com wrote:
The libffi library, headers and documentation are still installed, although
libffi provides separate releases for a long time. So do not
But the last time I checked, modern darwin defined
_GLIBCXX_USE_C99_MATH_TR1, no problems.
AFAICT this is true, but I think darwin10 was released in 2011
so I doubt it has any support for c++11.
Anyway, about the cstdlib issue the below makes available the new
functions in c_std/cstdlib
On 02/12/2013 01:47 PM, domi...@lps.ens.fr wrote:
But the last time I checked, modern darwin defined
_GLIBCXX_USE_C99_MATH_TR1, no problems.
AFAICT this is true, but I think darwin10 was released in 2011
so I doubt it has any support for c++11.
Anyway, about the cstdlib issue the below makes
Hey,
seems like that last of the scanf changes are in.
We're intercepting __isoc99_*scanf irrespective of the glibc version,
because (a) it does not hurt (and with the static runtime, even
interceptor itself is thrown out by the linker), and (b) user program
and tool's runtime can be built with
On 11/02/13 15:43, Christophe Lyon wrote:
Richard,
Thanks for your comments.
Here a new version with the changes you suggested.
Thanks for turning this around quickly. This is fine.
Ramana and I have discussed this and we're agreed that we'd like this to
go into 4.8. However, if any
On Tue, Feb 12, 2013 at 05:28:53PM +0400, Evgeniy Stepanov wrote:
Hey,
seems like that last of the scanf changes are in.
We're intercepting __isoc99_*scanf irrespective of the glibc version,
because (a) it does not hurt (and with the static runtime, even
interceptor itself is thrown out by
This fixes the code difference -O3 vs. -O3 -flto reported in PR56295.
For the -flto run we are confused by the extra MEM_REF we wrap every
global decl with in the IL (that is to support seamless and alias-correct
replacement of global decls with their prevailing decl). The following
un-wraps the
On Mon, Feb 11, 2013 at 06:15:05PM -0600, Aldy Hernandez wrote:
How does this look?
Looks good to me.
Jakub, what's this you mention in the PR about caching
__optimize__((3))? You also mention I shouldn't compare against
this_target_optabs, but default_target_optabs. But what if
We add global register vars into the LTO symtab processed by the
linker. That's of course bogus. Fixed with the following patch.
Bootstrap regtest running on x86_64-unknown-linux-gnu.
Richard.
2013-02-12 Richard Biener rguent...@suse.de
PR lto/56297
* lto-streamer-out.c
On Tue, Feb 12, 2013 at 03:19:37PM +0100, Dodji Seketeli wrote:
gcc/
* Makefile.in (asan.o): Add new dependency on hash-table.h
* asan.c (struct asan_mem_ref, struct mem_ref_hasher): New types.
(asan_mem_ref_init, asan_mem_ref_get_end, get_mem_ref_hash_table)
On 7 February 2013 11:38, Richard Biener rguent...@suse.de wrote:
This rewrites fix_loop_structure to be equivalent to loop
detection from scratch via flow_loops_find with re-using
of existing loop tree entries.
This addresses a few shortcomings of fix_loop_structure. First,
as shown by the
On Tue, 12 Feb 2013, Marcus Shawcroft wrote:
On 7 February 2013 11:38, Richard Biener rguent...@suse.de wrote:
This rewrites fix_loop_structure to be equivalent to loop
detection from scratch via flow_loops_find with re-using
of existing loop tree entries.
This addresses a few
On 12 February 2013 14:42, Richard Earnshaw rearn...@arm.com wrote:
On 11/02/13 15:43, Christophe Lyon wrote:
Richard,
Thanks for your comments.
Here a new version with the changes you suggested.
Thanks for turning this around quickly. This is fine.
Ramana and I have discussed this
Sorry, I'll send that again copying gcc-patches in this time...
-- Forwarded message --
From: Marcus Shawcroft marcus.shawcr...@gmail.com
Date: 12 February 2013 15:17
Subject: Re: [PATCH] Fix PR56181, rewrite fix_loop_structure
To: Richard Biener rguent...@suse.de
On 12
On Tue, 12 Feb 2013, Marcus Shawcroft wrote:
Sorry, I'll send that again copying gcc-patches in this time...
-- Forwarded message --
From: Marcus Shawcroft marcus.shawcr...@gmail.com
Date: 12 February 2013 15:17
Subject: Re: [PATCH] Fix PR56181, rewrite fix_loop_structure
Hi,
I've created a new branch, called c++-concepts for work being done
for C++ concepts (yes, again.) This new effort is unrelated to
previous ConceptGCC so I left alone the inactive branch c++0x-concepts.
The patch below has been committed.
Thanks,
-- Gaby
Index: svn.html
Jakub, what's this you mention in the PR about caching
__optimize__((3))? You also mention I shouldn't compare against
this_target_optabs, but default_target_optabs. But what if
this_target_optabs has changed? (See patch).
The reason for that is that this_target_optabs could at that point
Yes, this looks good. I've added some tests and committed to
compiler-rt. Thanks!
On Tue, Feb 12, 2013 at 5:46 PM, Jakub Jelinek ja...@redhat.com wrote:
On Tue, Feb 12, 2013 at 05:28:53PM +0400, Evgeniy Stepanov wrote:
Hey,
seems like that last of the scanf changes are in.
We're intercepting
This patch from Alan Modra fixes a section type conflict error. See also
http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02172.html
Test results on PowerPC without this patch:
http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg01351.html
Test results on PowerPC with this patch:
On Tue, Feb 12, 2013 at 09:58:38AM -0600, Aldy Hernandez wrote:
OK for trunk?
I'd still prefer Richard to chime in, I'm really not familiar enough
with MIPS switchable target stuff.
+/* Recompute the optabs. If they have changed, save the new set of
+ optabs in the optimization node
Although __int128_t is technically not an extended integer type because
we don't want to change intmax_t, it seems appropriate to me to give it
the same semantics as an extended integer type apart from that one aspect.
The only thing we weren't implementing is support for numeric literals,
Jakub Jelinek ja...@redhat.com writes:
On Tue, Feb 12, 2013 at 03:19:37PM +0100, Dodji Seketeli wrote:
gcc/
* Makefile.in (asan.o): Add new dependency on hash-table.h
* asan.c (struct asan_mem_ref, struct mem_ref_hasher): New types.
(asan_mem_ref_init, asan_mem_ref_get_end,
Jakub Jelinek ja...@redhat.com writes:
On Mon, Feb 11, 2013 at 06:15:05PM -0600, Aldy Hernandez wrote:
How does this look?
Looks good to me.
Jakub, what's this you mention in the PR about caching
__optimize__((3))? You also mention I shouldn't compare against
this_target_optabs, but
On Tue, Feb 12, 2013 at 05:28:11PM +0100, Dodji Seketeli wrote:
Thanks. Here is the updated patch that hopefully addresses your
comments.
Thanks, yes. Just please add
/* { dg-final { cleanup-tree-dump asan0 } } */
lines to all new tests after the last dg-final line in those tests and check
it
Rather than:
/* Change optabs if needed. */
if (TREE_OPTIMIZATION_OPTABS (opts))
this_target_optabs
= (struct target_optabs *) TREE_OPTIMIZATION_OPTABS (opts);
else
this_target_optabs = default_target_optabs;
I think it'd be
Richard Sandiford rdsandif...@googlemail.com writes:
Aldy Hernandez al...@redhat.com writes:
Rather than:
/* Change optabs if needed. */
if (TREE_OPTIMIZATION_OPTABS (opts))
this_target_optabs
= (struct target_optabs *) TREE_OPTIMIZATION_OPTABS (opts);
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56148
The patch was successfully bootstrapped and tested on x86/x86-64.
Committed as rev.195998.
2013-02-12 Vladimir Makarov vmaka...@redhat.com
PR inline-asm/56148
* lra-constraints.c
Aldy Hernandez al...@redhat.com writes:
Rather than:
/* Change optabs if needed. */
if (TREE_OPTIMIZATION_OPTABS (opts))
this_target_optabs
= (struct target_optabs *) TREE_OPTIMIZATION_OPTABS (opts);
else
this_target_optabs =
On 12 February 2013 15:38, Richard Biener rguent...@suse.de wrote:
On Tue, 12 Feb 2013, Marcus Shawcroft wrote:
Sorry, I'll send that again copying gcc-patches in this time...
-- Forwarded message --
From: Marcus Shawcroft marcus.shawcr...@gmail.com
Date: 12 February 2013
On Sun, Feb 10, 2013 at 12:29:41AM +0100, Marc Glisse wrote:
2013-02-07 Marek Polacek pola...@redhat.com
* c-parser.c (origtypes): Initialize to NULL.
Maybe mention PR44938? (there may be a dup or two)
Ok, I added that.
(I thought it was the name of the function that went inside
Hi,
This patch updates GCC 4.8 changes.html to mention Broadwell's
features, RTM and HLE support and fixed pre-reload scheduler.
OK to commit?
Thanks,
Igor
Index: htdocs/gcc-4.8/changes.html
===
RCS file:
Add class record_builder to ease construction of records and unions. Use it
in some appropriate places.
Nathan please review the vxworks changes.
tree.h
New class record_builder.
tree.c
Implement record_builder member functions.
asan.c
Change asan_global_struct to use
On Tue, Feb 12, 2013 at 08:26:37PM +0100, Marek Polacek wrote:
That's right, so I used something hopefully better this time around ;)
Ok for trunk now?
2013-02-12 Marek Polacek pola...@redhat.com
PR c/44938
* c-parser.c (c_parser_postfix_expression_after_primar): Initialize
On Tue, Feb 12, 2013 at 8:45 PM, Igor Zamyatin izamya...@gmail.com wrote:
This patch updates GCC 4.8 changes.html to mention Broadwell's
features, RTM and HLE support and fixed pre-reload scheduler.
OK to commit?
Thanks,
Igor
Index: htdocs/gcc-4.8/changes.html
On Tue, Feb 12, 2013 at 08:52:27PM +0100, Jakub Jelinek wrote:
On Tue, Feb 12, 2013 at 08:26:37PM +0100, Marek Polacek wrote:
That's right, so I used something hopefully better this time around ;)
Ok for trunk now?
2013-02-12 Marek Polacek pola...@redhat.com
PR c/44938
*
On 12/02/13 19:59, Marek Polacek wrote:
Thanks for noticing, I used auto-completing of words...
I'll fix it up and commit.
Marek
Why did this occur? I am curious.
Alec
We need to move for rvalue ref parms as well as value ones.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 29189e48dc7a62ff6a3a67984dbc64f4ada72862
Author: Jason Merrill ja...@redhat.com
Date: Tue Feb 12 12:56:52 2013 -0500
PR c++/56285
* method.c (add_one_base_init): Handle
This patch fixes a bad merge from google/integration to google/4_7.
Passes regression tests. Ok for google/4_7?
Thanks,
Teresa
2013-02-12 Teresa Johnson tejohn...@google.com
* coverage.c (coverage_init): Move test coverage handling
to the correct location within coverage_init
ok. The same problem exists in google/main too.
David
On Tue, Feb 12, 2013 at 1:38 PM, Teresa Johnson tejohn...@google.com wrote:
This patch fixes a bad merge from google/integration to google/4_7.
Passes regression tests. Ok for google/4_7?
Thanks,
Teresa
2013-02-12 Teresa Johnson
Ok, I will apply the same fix to google/main after appropriate testing.
Teresa
On Tue, Feb 12, 2013 at 1:54 PM, Xinliang David Li davi...@google.com wrote:
ok. The same problem exists in google/main too.
David
On Tue, Feb 12, 2013 at 1:38 PM, Teresa Johnson tejohn...@google.com wrote:
This
From: David Miller da...@redhat.com
Date: Fri, 16 Nov 2012 00:33:05 -0500 (EST)
From: Richard Sandiford rdsandif...@googlemail.com
Date: Mon, 29 Oct 2012 10:14:53 +
...given that the code is like you say written:
if (SHIFT_COUNT_TRUNCATED)
{
if (CONST_INT_P (op1)
On Tue, Feb 12, 2013 at 08:21:06PM +, Alec Teal wrote:
Why did this occur? I am curious.
It is caused by -fipa-cp-clone, which -O3 entails.
Marek
57 matches
Mail list logo