Hi
The below patch enables FMA4 for AMD bdver3 architectures.
make -k check passes.
Is it OK for upstream?
Regards
Ganesh
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index fb5b267..cbb5311 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2013-10-16 Ganesh Gopalasubramanian
On Wed, Oct 16, 2013 at 8:28 AM, Gopalasubramanian, Ganesh
ganesh.gopalasubraman...@amd.com wrote:
The below patch enables FMA4 for AMD bdver3 architectures.
make -k check passes.
+2013-10-16 Ganesh Gopalasubramanian ganesh.gopalasubraman...@amd.com
+
+ * config/i386/i386.c
On Wed, Oct 16, 2013 at 09:00:58AM +0200, Uros Bizjak wrote:
On Wed, Oct 16, 2013 at 8:28 AM, Gopalasubramanian, Ganesh
ganesh.gopalasubraman...@amd.com wrote:
The below patch enables FMA4 for AMD bdver3 architectures.
make -k check passes.
+2013-10-16 Ganesh Gopalasubramanian
Hello!
This patch to libgo fixes a dumb bug in which catching a signal via the
os/signal package could clobber the saved split-stack context, leading
to disaster if the signal arrived at the wrong time. Bootstrapped and
ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
On Wed, 2013-10-16 16:24:12 +1030, Alan Modra amo...@gmail.com wrote:
On Tue, Oct 15, 2013 at 02:45:23PM -0700, Sriraman Tallam wrote:
I committed this patch after making the above change.
/src/gcc-virgin/gcc/config/rs6000/rs6000.c: At global scope:
On Tue, 15 Oct 2013, Mike Stump wrote:
On Oct 15, 2013, at 12:45 AM, Richard Biener rguent...@suse.de wrote:
On Mon, 14 Oct 2013, Mike Stump wrote:
On Sep 25, 2012, at 8:00 AM, Richard Guenther rguent...@suse.de wrote:
2012-09-25 Richard Guenther rguent...@suse.de
PR lto/54625
On Tue, Oct 15, 2013 at 5:11 PM, Paulo Matos pma...@broadcom.com wrote:
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On
Behalf Of Jakub Jelinek
Sent: 15 October 2013 15:45
To: Paulo Matos
Cc: Richard Biener; Paolo Carlini;
On Tue, Oct 15, 2013 at 6:55 PM, Jeff Law l...@redhat.com wrote:
On 10/15/13 10:53, Jakub Jelinek wrote:
On Tue, Oct 15, 2013 at 10:50:39AM -0600, Jeff Law wrote:
I noticed that we're now including rtl.h and tm_p.h in
tree-ssa-reassoc.c, which seems wrong.
Isn't that required for
-Original Message-
From: Mike Stump [mailto:mikest...@comcast.net]
Sent: 14 October 2013 20:46
To: Kyrill Tkachov
Cc: Paulo Matos; gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] Fix PR58682
In this case, I've reviewed the license I suspect you have, and I suspect it
does
not have
Dear Tobias,
Your patch is fine for trunk.
Thanks
Paul
On 16 October 2013 00:38, Tobias Burnus bur...@net-b.de wrote:
As the test case (see also PR) showed, gfortran was rejecting:
subroutine list_move_alloc(self,item)
class(list_node),intent(inout) :: self
Hi,
The patch enhances ifcombine pass to recover some non short circuit
branches. Basically, it will do the following transformation:
Case 1:
if (cond1)
if (cond2)
==
if (cond1 cond2)
Case 2:
if (cond1)
goto L1
if (cond2)
goto L1
==
if (cond1 || cond2)
goto L1
Case
Hi!
On Mon, 14 Oct 2013 11:26:21 +0100, Kyrill Tkachov kyrylo.tkac...@arm.com
wrote:
This minuscule patch moves the dg-require-effective-target fopenmp line after
the other directives in the c-c++-common/cpp/openmp-define-3.c test.
Otherwise
dejagnu still proceeds to add -fopenmp to the
On Wed, Oct 16, 2013 at 7:59 AM, Jeff Law l...@redhat.com wrote:
On 07/08/13 11:45, Kai Tietz wrote:
Hello,
These passes are implementing type-demotion (type sinking into
statements) for some gimple statements. I limitted inital
implementation to the set of multiply, addition,
Hi,
we ICE on the C++11 version of the GNU visibility attribute on a
namespace. I suppose we want to accept it, then just use
get_attribute_name instead of TREE_PURPOSE. The bug report also asked
for an example in the docs of attribute on a namespace declaration.
Tested x86_64-linux.
Hi Thomas,
--- a/gcc/testsuite/c-c++-common/cpp/openmp-define-3.c
+++ b/gcc/testsuite/c-c++-common/cpp/openmp-define-3.c
@@ -1,6 +1,6 @@
-/* { dg-require-effective-target fopenmp } */
/* { dg-options -fopenmp } */
/* { dg-do preprocess } */
+/* { dg-require-effective-target fopenmp } */
The
PIng?
On 19 September 2013 18:21, Charles Baylis charles.bay...@linaro.org wrote:
Hi
Here is an updated version.
Changelog:
* gcc.dg/builtin-apply2.c: skip test on arm hardfloat ABI targets
* gcc.dg/tls/pr42894.c: Remove options, forcing -mthumb fails
with hardfloat, and
4.8.2 is already rolling, so too late for that.
Is 4.8 branch (gcc/branches/gcc-4_8-branch) open?
If yes, shall I commit these changes?
Regards
Ganesh
-Original Message-
From: Jakub Jelinek [mailto:ja...@redhat.com]
Sent: Wednesday, October 16, 2013 12:41 PM
To: Uros Bizjak
Cc:
Hi,
Resending filename-normalize patch to correct list gcc-patches@gcc.gnu.org.
All context, please, see below.
Regards
Vladimir Simonov
-Original Message-
From: Vladimir Simonov
Sent: Wednesday, October 16, 2013 12:52 PM
To: 'Jiri Dobry'; 'gcc-h...@gcc.gnu.org'
Subject: RE: MAX_PATH
Hi!
On Wed, 16 Oct 2013 10:54:48 +0100, Kyrill Tkachov kyrylo.tkac...@arm.com
wrote:
--- a/gcc/testsuite/c-c++-common/cpp/openmp-define-3.c
+++ b/gcc/testsuite/c-c++-common/cpp/openmp-define-3.c
@@ -1,6 +1,6 @@
-/* { dg-require-effective-target fopenmp } */
/* { dg-options -fopenmp }
This PR is about _Alignas in contexts like
char x[_Alignas (int) 20];
C grammar does not allow _Alignas to be in an array declarator, yet we
don't issue any warning or an error. This patch implements a pedwarn
for this. I'm however highly unsure whether we want pedwarn, error, or
normal warning
On Wed, 16 Oct 2013, Zhenqiang Chen wrote:
The patch enhances ifcombine pass to recover some non short circuit
branches. Basically, it will do the following transformation:
Case 1:
if (cond1)
if (cond2)
==
if (cond1 cond2)
Hello,
it might be more clear if you wrote (cond1 cond2) so
Hi,
bootstrap is currently broken, I'm going to test and commit the below if
everything goes well.
Thanks,
Paolo.
///
Index: pt.c
===
--- pt.c(revision 203698)
+++ pt.c(working copy)
@@ -12102,7
Hi,
after the discussion in the thread below I've committed the enabling
of LRA on ARM through the new option -mlra. Notice that we still rely
on reload until the regression has been resolved.
http://gcc.gnu.org/ml/gcc-patches/2013-10/msg00908.html
Thanks,
Yvan
2013-10-16 Yvan Roux
For vector unpack operations, the meaning of high and low is
reversed for little endian.
Bootstrapped and tested on powerpc64{,le}-unknown-linux-gnu with no
regressions. This fixes one test case for little endian
(gcc.dg/vect/vect-122.c).
Ok for trunk?
Thanks,
Bill
2013-10-16 Bill Schmidt
On Wed, 16 Oct 2013, Marek Polacek wrote:
This PR is about _Alignas in contexts like
char x[_Alignas (int) 20];
C grammar does not allow _Alignas to be in an array declarator, yet we
don't issue any warning or an error. This patch implements a pedwarn
for this. I'm however highly unsure
Hi,
On Tue, 15 Oct 2013, Richard Biener wrote:
On Sun, Oct 6, 2013 at 2:22 PM, Bernd Edlinger
bernd.edlin...@hotmail.de wrote:
How I should proceed with this patch, is it OK?
The latest version was posted at:
http://gcc.gnu.org/ml/gcc-patches/2013-09/msg00234.html
I have now
Hi Richard,
well I think I have now a solution for both of your comments on the
initial version of the portable volatility warning patch.
Furthermore I have integrated Sandra's comments.
Therefore I think it might be worth another try, if you don't mind.
Technically this patch is not dependent
Bah, bounced due to html or something. My latest email client upgrade
must have lost the 'plain text only' attribute..
On 10/16/2013 08:02 AM, Andrew MacLeod wrote:
More or less the same patch that I put in bugzilla
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58697
I've bootstrapped it and
On Tue, Oct 15, 2013 at 09:53:22AM -0600, Jeff Law wrote:
Martin put his entry in the wrong ChangeLog. (toplevel instead of
inside the gcc subdirectory).
Ouch, sorry and thanks for spotting and fixing it.
Martin
Fixed in the obvious way.
diff --git a/ChangeLog b/ChangeLog
index
Hi,
Just one question. You describe the pragma in the doco patch as:
+This pragma tells the compiler that the immediately following @code{for}
+loop can be executed in any loop index order without affecting the result.
+The pragma aids optimization and in particular vectorization as the
-Original Message-
From: Paolo Carlini [mailto:paolo.carl...@oracle.com]
Sent: 16 October 2013 12:54
To: Richard Biener; Paulo Matos
Cc: Jakub Jelinek; gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] tree_code_name wrapper
Hi,
bootstrap is currently broken, I'm going to test and
On Wed, Oct 16, 2013 at 12:24 AM, Uros Bizjak ubiz...@gmail.com wrote:
This patch to libgo fixes a dumb bug in which catching a signal via the
os/signal package could clobber the saved split-stack context, leading
to disaster if the signal arrived at the wrong time. Bootstrapped and
ran Go
I committed this patch to the website to document that the 4.8.2 release
includes full support for Go 1.1.2.
Ian
Index: gcc-4.8/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v
retrieving revision 1.122
diff
Speaking in terms of a patch:
Index: tree.h
===
--- tree.h (revision 203690)
+++ tree.h (working copy)
@@ -5184,14 +5184,11 @@ wi::int_traits const_tree::decompose (
/ HOST_BITS_PER_WIDE_INT);
I've committed initial support for shorthand constraints. This patch
adds support for parsing a concept-names as non-class names. When
introducing a template parameter, the concept name is transformed into
a constraint on the template parameter. Constrained parameters can
introduce type, non-type
On Wed, Oct 16, 2013 at 2:56 PM, Andrew MacLeod amacl...@redhat.com wrote:
Bah, bounced due to html or something. My latest email client upgrade must
have lost the 'plain text only' attribute..
On 10/16/2013 08:02 AM, Andrew MacLeod wrote:
More or less the same patch that I put in bugzilla
On Wed, Oct 16, 2013 at 8:22 AM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
For vector unpack operations, the meaning of high and low is
reversed for little endian.
Bootstrapped and tested on powerpc64{,le}-unknown-linux-gnu with no
regressions. This fixes one test case for little
Frederic Riss wrote:
Just one question. You describe the pragma in the doco patch as:
+This pragma tells the compiler that the immediately following @code{for}
+loop can be executed in any loop index order without affecting the result.
+The pragma aids optimization and in particular
The existing AArch64 implementation of preferred_reload_class when
called with a destination regclass STACK_REG returns the wider register
class GENERAL_REG. This patch changes the behavior such that we only
return a narrower class (or NO_REGS) and never return a wider class.
I'll leave
The existing aarch64 implementation of REGNO_REGCLASS classifies the
soft frame and arg registers as CORE_REGS. However either could be
eliminated against FP or SP, in case of the latter CORE_REGS is too
narrow. This patch changes the classification to POINTER_REGS a
superset of CORE_REGS
Hi,
To move a scalar char/short/int around in the vector registers there
is no such instruction as:
dup v0, v0.h[0]
But there is:
dup h0, v0.h[0]
(Alternately there is dup v0.4h, v0.h[0], but I don't think that
is what we are aiming for).
Fix the output template we are using to reflect
On 16 October 2013 15:58, James Greenhalgh james.greenha...@arm.com wrote:
Hi,
To move a scalar char/short/int around in the vector registers there
is no such instruction as:
dup v0, v0.h[0]
But there is:
dup h0, v0.h[0]
(Alternately there is dup v0.4h, v0.h[0], but I don't think that
On Wed, Oct 16, 2013 at 12:28:32PM +, Joseph S. Myers wrote:
On Wed, 16 Oct 2013, Marek Polacek wrote:
This PR is about _Alignas in contexts like
char x[_Alignas (int) 20];
C grammar does not allow _Alignas to be in an array declarator, yet we
don't issue any warning or an error.
On Tue, 2013-10-15 at 12:36 -0600, Jeff Law wrote:
On 09/24/13 11:49, David Malcolm wrote:
[...]
* gengtype-parse.c (struct_field_seq): Ignore access-control
keywords (public: etc).
OK.
Thanks; committed to trunk as r203708.
On Tue, 2013-10-15 at 13:19 +, Paulo Matos wrote:
-Original Message-
From: Jakub Jelinek [mailto:ja...@redhat.com]
Sent: 15 October 2013 10:51
To: Paulo Matos
Cc: Richard Biener; Paolo Carlini; gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] tree_code_name wrapper
On Tue,
On 10/15/13 12:16 PM, Jeff Law wrote:
On 10/10/13 10:25, Jakub Jelinek wrote:
On Thu, Oct 10, 2013 at 07:26:43AM -0700, Cesar Philippidis wrote:
This patch addresses an ICE when building qemu for a Mips target in
Yocto. Both gcc-trunk, gcc-4.8 and all of the targets are potentially
affected.
For me it worked with:
(add-hook 'c-mode-hook 'gcc-c-mode)
(defun gcc-c-mode()
;; set gnu style.
(c-set-style gnu)
;; TAB offset set to 2
(setq c-basic-offset 2)
;; Tabs should be 8
(setq indent-tabs-mode t)
)
Then I simply edit all my software as if it was GCC (haven't found a way
On 10/16/2013 09:55 AM, Richard Biener wrote:
Speaking in terms of a patch:
Index: tree.h
===
--- tree.h (revision 203690)
+++ tree.h (working copy)
@@ -5184,14 +5184,11 @@ wi::int_traits const_tree::decompose (
Paulo Matos pma...@broadcom.com writes:
(haven't found a way to say : 'only use this for GCC sources')
You could put it in .dir-locals.el, or put a hook on find-file-hook that
looks at buffer-file-name.
Andreas.
--
Andreas Schwab, SUSE Labs, sch...@suse.de
GPG Key fingerprint = 0196 BAD8
Hello,
It seems that gang of AVX* patterns were copy and
pasted from SSE, however as far as they are NDD,
we may remove corresponding expands which sort operands.
ChangeLog:
* config/i386/sse.md (vec_widen_umult_even_v8si): Remove expand,
make insn visible, remove redundant
Hello,
On 15 Oct 08:46, Richard Henderson wrote:
On 10/09/2013 03:31 AM, Kirill Yukhin wrote:
+ rtx op1 = operands[1];
+ if (REG_P (op1))
+op1 = gen_rtx_REG (V16HImode, REGNO (op1));
+ else
+op1 = gen_lowpart (V16HImode, op1);
The IF case is incorrect. You need to use
This patch makes the PPC the first public target to use the
CONST_WIDE_INT representation rather than CONST_DOUBLES to represent
larger integers. There are two parts to this patch. The changes to
genrecog.c represent fixes that would be needed for any port that
supports CONST_WIDE_INTS
On Wed, Oct 16, 2013 at 6:06 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
It seems that gang of AVX* patterns were copy and
pasted from SSE, however as far as they are NDD,
we may remove corresponding expands which sort operands.
ChangeLog:
* config/i386/sse.md
Andreas You could put it in .dir-locals.el, or put a hook on
Andreas find-file-hook that looks at buffer-file-name.
We checked in a .dir-locals.el for gdb. I recommend it for GCC as well.
The gdb one is appended. You'll at least want to edit the bugzilla URL
to suit.
To make this work, just
On 10/16/13 03:31, Richard Biener wrote:
I see two primary effects of type sinking.
Note it was called type demotion ;)
;) It's a mental block of mine; it's been called type hoisting/sinking
in various contexts and I see parallels between the code motion
algorithms and how the type
On Wed, Oct 16, 2013 at 6:06 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
It seems that gang of AVX* patterns were copy and
pasted from SSE, however as far as they are NDD,
we may remove corresponding expands which sort operands.
OTOH, I have some second thoughts on removing AVX2
On 10/16/2013 09:07 AM, Kirill Yukhin wrote:
I suspect gen_lowpart is bad turn when reload is completed, as
far as it can create new pseudo. gen_lowpart () may call
gen_reg_rtx (), which contain corresponging gcc_assert ().
False. gen_lowpart is perfectly safe post-reload.
Indeed, taking the
On 10/16/2013 09:47 AM, Uros Bizjak wrote:
On Wed, Oct 16, 2013 at 6:06 PM, Kirill Yukhin kirill.yuk...@gmail.com
wrote:
It seems that gang of AVX* patterns were copy and
pasted from SSE, however as far as they are NDD,
we may remove corresponding expands which sort operands.
OTOH, I
On Wed, 16 Oct 2013, Marek Polacek wrote:
Regtested/bootstrapped on x86_64-linux, ok for trunk?
2013-10-16 Marek Polacek pola...@redhat.com
PR c/58267
c/
* c-parser.c (c_parser_declspecs): Add alignspec_ok parameter.
Document syntax of the array-declarator.
I've committed the following to dmalcolm/jit:
Convert local creation to take a function rather than a context.
Simplify the API by eliminating the separate local type in favor of
just using the lvalue type, as we do for globals. As well as reducing
the number of API entrypoints, this simplifies
On Oct 16, 2013, at 8:47 AM, Kenneth Zadeck zad...@naturalbridge.com wrote:
To avoid code quality wreckage we have to implement a different way
of allocating the 'scratch' storage of wide_int_ref_storage
I want richard and mike to be the people who respond to the next point.I
am not a
I committed this patch to the 4.8 branch to bring in the last set of Go
1.1.2 bug fixes and patches. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu.
Ian
2013-10-16 Ian Lance Taylor i...@google.com
Bring in from mainline:
2013-10-11 Chris Manghane
On 10/02/13 16:53, Jason Merrill wrote:
On 08/27/2013 04:03 PM, Aldy Hernandez wrote:
+ else if (!TREE_TYPE (e) || !TREE_CONSTANT (e)
+ || !INTEGRAL_TYPE_P (TREE_TYPE (e)))
+cp_parser_error (parser,
+ step size must be an integer constant);
The sequence of events here can be summarized as shrink-wrapping causes
the i386 backend to do something that confuses alias analysis. The
miscompilation is that two instructions are swapped by the scheduler
when they shouldn't be, due to an incorrect reg_base_value.
The miscompiled function has
On 10/16/13 02:21, Richard Biener wrote:
On Tue, Oct 15, 2013 at 6:55 PM, Jeff Law l...@redhat.com wrote:
On 10/15/13 10:53, Jakub Jelinek wrote:
On Tue, Oct 15, 2013 at 10:50:39AM -0600, Jeff Law wrote:
I noticed that we're now including rtl.h and tm_p.h in
tree-ssa-reassoc.c, which seems
On 10/16/13 09:34, Cesar Philippidis wrote:
On 10/15/13 12:16 PM, Jeff Law wrote:
On 10/10/13 10:25, Jakub Jelinek wrote:
On Thu, Oct 10, 2013 at 07:26:43AM -0700, Cesar Philippidis wrote:
This patch addresses an ICE when building qemu for a Mips target in
Yocto. Both gcc-trunk, gcc-4.8 and
On 10/16/2013 01:48 PM, Aldy Hernandez wrote:
On 10/02/13 16:53, Jason Merrill wrote:
Can't the step size be a value-dependent expression like a template
non-type parameter?
Jason, I'm not sure. The standard says:
The conditional-expression in a simd-linear-step shall either
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Aldy Hernandez
Sent: Wednesday, October 16, 2013 1:49 PM
To: Jason Merrill
Cc: Richard Henderson; gcc-patches; Iyer, Balaji V
Subject: Re: PING: Fwd: Re: [patch] implement
On Oct 16, 2013, at 9:26 AM, Tom Tromey tro...@redhat.com wrote:
Andreas You could put it in .dir-locals.el, or put a hook on
Andreas find-file-hook that looks at buffer-file-name.
We checked in a .dir-locals.el for gdb. I recommend it for GCC as well.
I've copied the gdb one and updated it
On Wed, Oct 16, 2013 at 7:01 PM, Richard Henderson r...@redhat.com wrote:
On 10/16/2013 09:47 AM, Uros Bizjak wrote:
On Wed, Oct 16, 2013 at 6:06 PM, Kirill Yukhin kirill.yuk...@gmail.com
wrote:
It seems that gang of AVX* patterns were copy and
pasted from SSE, however as far as they are
Kenneth Zadeck zad...@naturalbridge.com writes:
I note that we immediately return in the above case, so
if (precision xprecision + HOST_BITS_PER_WIDE_INT)
{
len = wi::force_to_size (scratch, val, len, xprecision,
precision, UNSIGNED);
return
I've merged revision 203713 from the GCC 4.8 branch to the gccgo branch.
Ian
On Mon, Oct 14, 2013 at 08:59:50PM -0400, David Edelsohn wrote:
Please use VMX instead of AV, not because I am pedantic about
IBM's name for the feature, but AV is meaningless to me and anyone
else reading the code. If you need a three-letter name, use VMX.
Okay with that change.
I can
On Tue, Oct 15, 2013 at 3:39 PM, Teresa Johnson tejohn...@google.com wrote:
On Tue, Oct 15, 2013 at 2:57 PM, Jan Hubicka hubi...@ucw.cz wrote:
On Tue, Oct 15, 2013 at 2:03 PM, Jan Hubicka hubi...@ucw.cz wrote:
Yes, that would work. So let's discard this patch because the fix for
comdat can
Mike == Mike Stump mikest...@comcast.net writes:
Mike I've tested it to ensure everything works as expected, except I
Mike didn't see exactly how to test the bug-reference… If someone knows
Mike how to test that, I'd appreciate it. If I understand the net
Mike result of this, it should make the
On Mon, Oct 14, 2013 at 08:51:15PM -0400, David Edelsohn wrote:
As Ken Zadeck and Richard Sandiford noticed this morning, the F
constraint seems to be a cut-and-paste error because the F
constraint corresponds to an immediate scalar or vector float value,
which never will match TImode. Should
On Wed, 16 Oct 2013, Jeff Law wrote:
What's the policy on the GDFL stuff. I've tried so hard to avoid having to
worry about that rats nest that I have no idea what our policy is. Basically I
just want to take the old docs for BRANCH_COST and re-use those. Is that
considered kosher with all
On 10/15/13 15:30, Wei Mi wrote:
Aren't you just trying to see if we have a comparison feeding the
conditional jump and if they're already adjacent? Do you actually need to
get the condition code regs to do that test?
Yes, I am trying to see if we have a comparison feeding the
conditional
On 10/11/13 11:23, Andreas Krebbel wrote:
On 10/10/13 18:41, Jeff Law wrote:
On 10/10/13 04:00, Andreas Krebbel wrote:
On 09/10/13 21:46, Jeff Law wrote:
On 08/21/13 03:21, Andreas Krebbel wrote:
[RFC] Allow functions calling mcount before prologue to be leaf functions
Gerald == Gerald Pfeifer ger...@pfeifer.com writes:
Gerald % mkdir /scratch2/tmp/gerald/OBJ-1014-1920
Gerald % cd /scratch2/tmp/gerald/OBJ-1014-1920
Gerald % $GCC_SOURCE/configure --prefix=/home/gerald/something
Gerald % gmake bootstrap-lean
Gerald % gmake install
Gerald Wait a
On 10/16/2013 02:45 PM, Richard Sandiford wrote:
Kenneth Zadeck zad...@naturalbridge.com writes:
I note that we immediately return in the above case, so
if (precision xprecision + HOST_BITS_PER_WIDE_INT)
{
len = wi::force_to_size (scratch, val, len, xprecision,
On 16/10/13 19:32, Mike Stump wrote:
Committed revision 203715.
Awesome!
Can I backport your patch to 4_8?
--
PMatos
On Wed, Oct 16, 2013 at 3:00 PM, Michael Meissner
meiss...@linux.vnet.ibm.com wrote:
On Mon, Oct 14, 2013 at 08:59:50PM -0400, David Edelsohn wrote:
Please use VMX instead of AV, not because I am pedantic about
IBM's name for the feature, but AV is meaningless to me and anyone
else reading the
On Wed, Oct 16, 2013 at 3:08 PM, Michael Meissner
meiss...@linux.vnet.ibm.com wrote:
I'll try n. It probably should be something that allows CONST_DOUBLE ints
as
well as CONST_INT.
The 'n' constraint matches CONST_INT and CONST_DOUBLE.
- David
On 09/11/13 12:18, Iyer, Balaji V wrote:
Hello Everyone, Couple weeks back, I had submitted a patch for review
that will implement Cilk keywords (_Cilk_spawn and _Cilk_sync) into
the C compiler. I recently finished C++ implementation also. In this
email, I am attaching 2 patches: 1 for C (and
On Tue, Oct 15, 2013 at 10:54 PM, Alan Modra amo...@gmail.com wrote:
On Tue, Oct 15, 2013 at 02:45:23PM -0700, Sriraman Tallam wrote:
I committed this patch after making the above change.
/src/gcc-virgin/gcc/config/rs6000/rs6000.c: At global scope:
Early exit from cp_write_global_declarations in the PCH case completely
broke -fdump-translation-unit; the simple fix is just to do the dump on
the PCH path as well.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.8.
commit 106091d492179afa32d3042c4c74fb2ae6c440f4
Author: Jason Merrill
If we're deducing an array type from an initializer list, we can also
deduce the bound.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit c1effcb5fe5b63628a4a7ef75c96d9415ee18f20
Author: Jason Merrill ja...@redhat.com
Date: Mon Sep 23 06:15:25 2013 -0500
Core 1591
* pt.c
It's detected by `clang++ -g -std=c++11 -fsanitize=undefined` but not
detected by g++ with flags `g++ -g -std=c++11 -fsanitize=undefined
-lpthread`. Am I on the right way?
-m32 and -m64 tested :)
Thanks!
--
Tim Shen
a.patch
Description: Binary data
Printing noexcept (true) is redundant.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 5305dc0cdb0736a0c56ce3d126b4baad0521f096
Author: Jason Merrill ja...@redhat.com
Date: Thu Oct 3 15:35:11 2013 -0400
* error.c (dump_exception_spec): Print noexcept rather than
noexcept
While reviewing a recent patch I noticed that we weren't using
attribute_takes_identifier_p here. Fixed.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 021fe309badd55090a5bc212c44d24b7c23f37af
Author: Jason Merrill ja...@redhat.com
Date: Fri Oct 11 10:38:23 2013 -0400
* pt.c
On Oct 16, 2013, at 1:51 PM, Paulo J. Matos pa...@matos-sorge.com wrote:
On 16/10/13 19:32, Mike Stump wrote:
Committed revision 203715.
Awesome!
Can I backport your patch to 4_8?
First, we like to wait and let patches bake on mainline before considering back
porting, this has no bake
On 10/16/2013 11:48 PM, Tim Shen wrote:
It's detected by `clang++ -g -std=c++11 -fsanitize=undefined` but not
detected by g++ with flags `g++ -g -std=c++11 -fsanitize=undefined
-lpthread`. Am I on the right way?
-m32 and -m64 tested :)
Thank you!
Paolo.
On Wed, Oct 16, 2013 at 2:02 AM, Richard Biener rguent...@suse.de wrote:
On Tue, 15 Oct 2013, Cong Hou wrote:
Thank you for your reminder, Jeff! I just noticed Richard's comment. I
have modified the patch according to that.
The new patch is attached.
(posting patches inline is easier for
On Oct 16, 2013, at 12:03 PM, Tom Tromey tro...@redhat.com wrote:
Visit ChangeLog.
Find a reference to a PR.
M-x bug-reference-mode
Click or C-c RET on the PR text
Tested, confirmed it works as expected; thanks.
On Wed, Oct 16, 2013 at 6:13 PM, Paolo Carlini paolo.carl...@oracle.com wrote:
Thank you!
Committed.
Thanks!
--
Tim Shen
On 10/17/2013 12:29 AM, Tim Shen wrote:
On Wed, Oct 16, 2013 at 6:13 PM, Paolo Carlini paolo.carl...@oracle.com wrote:
Thank you!
Committed.
By the way, please feel free to prepare a reduced testcase for the
-fsanitize people (Marek, Jakub?)
Thanks,
Paolo.
On Wed, Oct 16, 2013 at 6:33 PM, Paolo Carlini paolo.carl...@oracle.com wrote:
By the way, please feel free to prepare a reduced testcase for the
-fsanitize people (Marek, Jakub?)
Here it is.
And we should get undefined behaviors before this commit(r203732).
--
Tim Shen
split.cc
On 10/17/2013 12:39 AM, Tim Shen wrote:
On Wed, Oct 16, 2013 at 6:33 PM, Paolo Carlini paolo.carl...@oracle.com wrote:
By the way, please feel free to prepare a reduced testcase for the
-fsanitize people (Marek, Jakub?)
Here it is.
And we should get undefined behaviors before this
Building newlib uncovered a few invalid assumptions... Committed.
* config/rl78/rl78.c (rl78_alloc_address_registers_macax): Verify
op is a REG before checking REGNO.
(rl78_alloc_physical_registers): Verify pattern is a SET before
checking SET_SRC.
Index:
1 - 100 of 116 matches
Mail list logo