On Fri, 10 May 2013, Jakub Jelinek wrote:
Hi!
I've apparently missed one spot, where store of zero value was assumed,
but not actually verified. Fixed thusly, bootstrapped/regtested on
x86_64-linux and i686-linux, ok for trunk/4.8/4.7?
Ok.
Thanks,
Richard.
2013-05-10 Jakub Jelinek
On Fri, May 10, 2013 at 8:27 PM, Carrot Wei car...@google.com wrote:
Hi
In function dw2_output_indirect_constant_1 a new var decl is created. Only
When the variable is not PUBLIC it is allocated static storage. Does anybody
know why the variable is not allocated static storage by marking
On Sat, May 11, 2013 at 1:22 AM, Steven Bosscher stevenb@gmail.com wrote:
Hello,
This unch_memory in struct resources is a left-over from
RTX_UNCHANGING_P, but it looks like the change-over to MEM_READONLY_P
was done incorrectly: The resource_conflicts_p code now reports
conflicts for
On Sat, Apr 13, 2013 at 08:21:46PM +0200, Steven Bosscher wrote:
* cfgrtl.c (cfg_layout_merge_blocks): Revert r184005, implement
correct fix by moving header and footer insn to the footer of
the merged basic block. Clear BB_END of the merged-away block.
Unfortunately
On Sat, May 11, 2013 at 1:41 AM, DJ Delorie d...@redhat.com wrote:
Note that I had to make a few changes (fixes?) in the MI portions of
gcc to avoid problems I encountered, I don't know if these changes are
correct or if there are better ways to avoid those cases. Those
In any case,
On Sun, May 12, 2013 at 2:04 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
this patch passes bootstrap+testsuite on x86_64-linux-gnu. When moving
uniform_vector_p, I only added the gcc_assert. Note that the fold_binary
patch helps for constant vectors, but not for {n,n,n,n}, which will
On Mon, May 13, 2013 at 5:45 AM, Kugan
kugan.vivekanandara...@linaro.org wrote:
Hi,
This patch removes some of the redundant sign/zero
extensions using value ranges informations generated by VRP.
When GIMPLE_ASSIGN stmts with LHS type smaller than word is expanded to
rtl, if we can prove
On 05/06/2013 05:56 PM, Jason Merrill wrote:
On 05/06/2013 08:46 AM, Florian Weimer wrote:
On 05/06/2013 02:39 PM, Jason Merrill wrote:
On 05/06/2013 05:46 AM, Florian Weimer wrote:
Nice, this is simpler than expected. However, it makes the call sites
even more bloated.
Hmm, perhaps the
On Sat, May 11, 2013 at 1:39 PM, Jan Hubicka hubi...@ucw.cz wrote:
But
glancing over the the dumps, I see many of them just have different
name
spaces. Do we even attempt to merge namespace_decl? How types from
same
namespaces in different units are supposed to match?
We do not merge
Thank you Uros!
Committed r198820.
Regards
Ganesh
-Original Message-
From: Uros Bizjak [mailto:ubiz...@gmail.com]
Sent: Tuesday, May 07, 2013 6:22 PM
To: Gopalasubramanian, Ganesh
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH,i386] Default alignment for AMD BD and BT
On Tue, May
On 05/09/13 07:02, Nathan Sidwell wrote:
On 05/08/13 18:47, Jan Hubicka wrote:
Thinking about it again, isn't decl_replaceable_p the thing you are looking for
here?
that looks promising. I'll try !decl_replaceable_p in the section anchor hook.
It does indeed seem to be the right
On 05/09/13 07:02, Nathan Sidwell wrote:
On 05/08/13 18:47, Jan Hubicka wrote:
Thinking about it again, isn't decl_replaceable_p the thing you are looking
for
here?
that looks promising. I'll try !decl_replaceable_p in the section anchor
hook.
It does indeed seem to be the right
On Sat, May 11, 2013 at 11:38 AM, Marc Glisse marc.gli...@inria.fr wrote:
Second try.
I removed the fold_single_bit_test thing (I thought I'd handle it, so I
started by the easy part, and never did the rest).
Adapting invert_truthvalue_loc for vectors, I thought: calling
fold_truth_not_expr
Hi,
This patch refactors the simd_mov split and fixes a few coding style issues.
Tested successfully on a full aarch64-elf regression run.
OK for trunk?
Thanks
Sofiane
aarch64-refactor-simd-mov.diff
Description: Binary data
This fixes a virtual SSA updating problem with sinking clobbers.
Namely when sinking into a block with multiple predecessors and
no virtual use we lack a convenient PHI node that serves as a
merge of the virtual operands from the predecessors. The following
patch gives up sinking clobbers in
On Sat, 11 May 2013, Jakub Jelinek wrote:
On Sat, May 11, 2013 at 09:05:52AM +0200, Jakub Jelinek wrote:
Seems that we ought to have a testcase, even though it probably
means scanning the tree dumps to pick up the undefined behaviour.
Approved with a testcase.
I have added lots of
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
As described in the PR, amd64-pc-solaris2.1[01] Ada bootstrap was failing
for some time. It has turned out that this patch is the culprit:
2013-04-23 Eric Botcazou ebotca...@adacore.com
Pascal Obryo...@adacore.com
On Mon, 13 May 2013, Richard Biener wrote:
On Sat, May 11, 2013 at 11:38 AM, Marc Glisse marc.gli...@inria.fr wrote:
Second try.
I removed the fold_single_bit_test thing (I thought I'd handle it, so I
started by the easy part, and never did the rest).
Adapting invert_truthvalue_loc for
Hi
The patch enables FSGSBASE instruction generation for AMD bdver3 architectures.
make -k check passes.
Is it OK for upstream?
Regards
Ganesh
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 198821)
+++ gcc/ChangeLog
On Mon, May 13, 2013 at 1:54 PM, Gopalasubramanian, Ganesh
ganesh.gopalasubraman...@amd.com wrote:
The patch enables FSGSBASE instruction generation for AMD bdver3
architectures.
make -k check passes.
Is it OK for upstream?
OK. Please also check for missing PTA_PRFCHW and PTA_FXSR for AMD
On 05/09/2013 06:41 PM, Jason Merrill wrote:
At the last C++ standards meeting, we agreed to add VLAs to the
language. But they're significantly different from GNU/C99 VLAs: you
can't form a pointer to a VLA, or take its sizeof, or really anything
other than directly use it. We also need to
Hi,
this patch makes i386.c to use correct predicates for hot/cold decisions. It
also makes
mode-switching to set correct RTL profile when emitting the initialization code.
Honza
Bootstrapped/regtested x86_64-linux, comitted.
* mode-switching.c (optimize_mode_switching): Set correct
Hi!
As expected, the C FE in this case was easier than C++, for
#pragma omp declare reduction it will be the other way around, C
will be much harder.
Committed to gomp-4_0-branch.
2013-05-13 Jakub Jelinek ja...@redhat.com
* c-tree.h (c_finish_omp_declare_simd): New prototype.
On Mon, May 13, 2013 at 7:25 AM, Florian Weimer fwei...@redhat.com wrote:
On 05/09/2013 06:41 PM, Jason Merrill wrote:
At the last C++ standards meeting, we agreed to add VLAs to the
language. But they're significantly different from GNU/C99 VLAs: you
can't form a pointer to a VLA, or take
On 05/13/2013 03:06 PM, Gabriel Dos Reis wrote:
This whole feature seems rather poorly designed to me. The code size
increase due to official VLA support in C++11y might come a bit as a
surprise. But rereading N3639, there's no way around it, at least for
expressions of signed types.
I think
Index: varasm.c
===
--- varasm.c (revision 198771)
+++ varasm.c (working copy)
@@ -6582,10 +6582,18 @@ default_use_anchors_for_symbol_p (const_
{
/* Don't use section anchors for decls that might be defined by
On 05/11/2013 01:29 PM, Steven Bosscher wrote:
Hello,
This just removes one unused function, and moves two functions from
emit-rtl.c to reorg.c which is the only place where they're used.
Will commit in a few days, barring objections.
Ciao!
Steven
* rtl.h (next_label,
friendly ping
On Mon, Apr 22, 2013 at 5:23 PM, Evgeniy Stepanov euge...@google.com wrote:
Hi,
this patch disables rdrand in c++11/random.cc when building with Clang
compiler. Current Clang misses a number of definitions needed to build
that.
Is it OK for google/gcc-4_8 and google/main (or
OK
/Marcus
On 13 May 2013 11:38, Sofiane Naci sofiane.n...@arm.com wrote:
Hi,
This patch refactors the simd_mov split and fixes a few coding style issues.
Tested successfully on a full aarch64-elf regression run.
OK for trunk?
Thanks
Sofiane
2013/5/13 Steven Bosscher stevenb@gmail.com:
On Sun, May 12, 2013 at 8:56 PM, Gerald Pfeifer wrote:
On Fri, 10 May 2013, DJ Delorie wrote:
Index: MAINTAINERS
===
+msp430 port DJ Delorie d...@redhat.com
re-sending the patch
On Mon, May 13, 2013 at 5:54 PM, Evgeniy Stepanov euge...@google.com wrote:
friendly ping
On Mon, Apr 22, 2013 at 5:23 PM, Evgeniy Stepanov euge...@google.com wrote:
Hi,
this patch disables rdrand in c++11/random.cc when building with Clang
compiler. Current Clang
On 05/13/2013 09:09 AM, Florian Weimer wrote:
And usually I'm in that crowd as well. But in this case, we add a check
which only covers a tiny fraction of the problem. It's like bounds
checking for arrays which only fails if the index is at least twice as
large as the array length, IMHO.
The
On 13/5/13 11:15 AM, Kugan wrote:
Hi,
Ping this patch by Chung-Lin.
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01179.html
This patch allows lr registers to be used in leaf functions for ARM.
There were some concerns about performance regression in thumb2 mode for
CoreMark. However,
Can someone please review this patch for us?
Thanks,
Balaji V. Iyer.
-Original Message-
From: Iyer, Balaji V
Sent: Monday, May 06, 2013 11:32 AM
To: Joseph S. Myers
Cc: 'Aldy Hernandez'; 'gcc-patches'
Subject: RE: [patch] cilkplus: Array notation for C patch
Attached, please
On Sun, May 12, 2013 at 02:04:52PM +0200, Marc Glisse wrote:
this patch passes bootstrap+testsuite on x86_64-linux-gnu. When
moving uniform_vector_p, I only added the gcc_assert. Note that the
fold_binary patch helps for constant vectors, but not for {n,n,n,n},
which will require some help in
Hi!
On the following testcase, the widen multiplies etc. is introduced so late
that no forwprop or ccp follows it. The WIDEN_MULT_EXPR tries hard not to
use widening multiply if both arguments are INTEGER_CSTs, but if they
are e.g. SSA_NAMEs that expand_normal into CONST_INT, we can still ICE.
On Fri, May 10, 2013 at 07:15:38PM +0200, Jan Hubicka wrote:
It seems to me that it is not different from normalizing reg-10 into reg+(-10)
we do for years (and for good reason). It is still target preference when use
add and when sub to perform the arithmetic, but it makes sense to keep
On Mon, 13 May 2013, Jakub Jelinek wrote:
On Sun, May 12, 2013 at 02:04:52PM +0200, Marc Glisse wrote:
this patch passes bootstrap+testsuite on x86_64-linux-gnu. When
moving uniform_vector_p, I only added the gcc_assert. Note that the
fold_binary patch helps for constant vectors, but not for
On Mon, May 13, 2013 at 06:46:00PM +0200, Marc Glisse wrote:
-O is enough. Could you commit that please? Or I can do it in a
couple hours.
Done.
Jakub
On Mon, May 13, 2013 at 6:43 PM, Jakub Jelinek ja...@redhat.com wrote:
On Fri, May 10, 2013 at 07:15:38PM +0200, Jan Hubicka wrote:
It seems to me that it is not different from normalizing reg-10 into
reg+(-10)
we do for years (and for good reason). It is still target preference when
use
On Wed, 8 May 2013, Ian Lance Taylor wrote:
+#ifdef IN_GCC
Where is IN_GCC defined?
I've amended configure.ac to provide a new configuration flag
('--enable-standalone') and add -DIN_GCC to EXTRA_FLAGS unless that flag is
given. Previously I've misread grep output and thought that IN_GCC is
Can you add that (partial int modes have fewer bits than int modes)
as verification to genmodes.c:make_partial_integer_mode?
I could, but it would be a no-op for PARTIAL_INT_MODE()
I wonder if this should not use GET_MODE_PRECISION - after all it is
the precision that determines whether we
Please find below an updated patch. Let me know if ok to commit.
Yup, this is OK to commit. Thanks!
Regards,
Kaushik
2013-05-13 Kaushik Phatak kaushik.pha...@kpitcummins.com
* config/rl78/rl78.md (mulsi3_g13): Add additional 'nop' required
in multiply-accumulate
Hello!
This PR again exposes the problem when STOS instruction is generated
from the combine pass. The insn is not generated from corresponding
expander, so ix86_current_function_needs_cld flag never gets set.
Consequently, the CLD insn is not emitted in the prologue.
The problematic combination
First, it adds a missing -std=f95 check for Fortran 2003's BIND(C)
statement.
Secondly, it honors the COMMON identifier changes of Fortran 2008. In
Fortran 2003, one has:
The name of a program unit, common block, or external procedure is a
global identifier and shall not be the same as the
All the gcc.c-torture/compile/limits-externdecl.c currently give an
error: size of array is too large, followed by an ICE in
avr_encode_section_info, which goes on to try to find the address space
of error_mark_node.
Given the size of the array, it makes sense for the test to give an error
where
So, it turns out that comm on macosx does't like long lines. Don't know why
that is, kinda unfortunate, but life goes on.
2013-05-13 Mike Stump mikest...@comcast.net
* compare_tests: Limit lines to 2000 characters as comm on Mac OS
X 10.8.3 doesn't like long lines (those 2055
The MV testing support includes 3 logical parts:
1) runtime APIs to check mocked CPU types and features
(__builtin_mock_cpu_supports ..)
2) runtime APIs to do CPU mocking;
3) compile time option to do lazy dispatching (instead of using IFUNC).
3) can be used to also support target without IFUNC
OK for google branches.
On Thu, May 9, 2013 at 1:40 PM, Han Shen(沈涵) shen...@google.com wrote:
Hi, I'm to backport trunk patch @198547 for pr target/56732 to google
branch google/gcc-4_8.
This patch fixes arm ICE.
Ok for google/gcc-4_8?
[patch attached]
H.
I would need a way to use GS segment register instead of FS for x86-64 for
target RDOS since RDOS cannot use FS for TLS. It seems like the code related
to this is concentrated to two different places:
The gcc/config/i386/i386.c:
11677:seg = TARGET_64BIT ? SEG_FS : SEG_GS;
3 places that hadn't yet been updated to handle ref-qualifiers.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.8.
commit e4f9e32f7e7b9e508f713ac15a9edbd275d4bd85
Author: Jason Merrill ja...@redhat.com
Date: Mon May 13 10:45:41 2013 -0400
PR c++/57252
* decl.c (decls_match):
If we don't like the designator, we should fail pleasantly.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.8.
commit be2b8cf9756a4ca4be6583f65fc4388ada2a5b77
Author: Jason Merrill ja...@redhat.com
Date: Mon May 13 13:26:12 2013 -0400
PR c++/57041
* decl.c (reshape_init_class):
Hi,
Is anyone able to review the below please (original patch attached to
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56780 and first posted at
http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00167.html
Thanks,
Matt.
On Wed, 2013-04-03 at 15:03 +0100, Matt Burgess wrote:
Hi,
Please find
In this testcase, since one of the overloads takes a pointer, we need to
check whether the argument is a valid null pointer constant. In C++11
only an integer literal can be a null pointer constant of integral type,
but in C++03 we need to handle other integer constant expressions. Here
we
On Mon, May 13, 2013 at 03:25:11PM -0400, Jason Merrill wrote:
In this testcase, since one of the overloads takes a pointer, we
need to check whether the argument is a valid null pointer constant.
In C++11 only an integer literal can be a null pointer constant of
integral type, but in C++03 we
Here is the latest version of my SSA optimization pass to do the switch
statement optimization described in PR 54742 (core_state_transition from
coremark).
I have tested this optimization with a x86 bootstrap and GCC test run
with no errors and tested the MIPS cross compiler with no errors.
On 05/13/2013 02:16 PM, Steve Ellcey wrote:
Here is the latest version of my SSA optimization pass to do the switch
statement optimization described in PR 54742 (core_state_transition from
coremark).
I have tested this optimization with a x86 bootstrap and GCC test run
with no errors and tested
Alexander == Alexander Monakov amona...@ispras.ru writes:
Alexander In GCC's own dwarf2.h as enum tags; dwarf.h uses anonymous enums.
I'm curious to know which dwarf2.h you are testing against and/or intend
to support. I think there is more than one.
Tom
On Tue, May 14, 2013 at 12:45 AM, Tom Tromey tro...@redhat.com wrote:
Alexander == Alexander Monakov amona...@ispras.ru writes:
Alexander In GCC's own dwarf2.h as enum tags; dwarf.h uses anonymous enums.
I'm curious to know which dwarf2.h you are testing against and/or intend
to support. I
On 05/13/2013 09:39 AM, Jakub Jelinek wrote:
2013-05-13 Jakub Jelinek ja...@redhat.com
PR middle-end/57251
* expr.c (expand_expr_real_2) case WIDEN_MULT_EXPR: Handle
the case when both op0 and op1 have VOIDmode.
* gcc.dg/torture/pr57251.c: New test.
Ok.
r~
Ping.
On Thu, May 9, 2013 at 2:20 PM, Sriraman Tallam tmsri...@google.com wrote:
cc:Diego
On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Thu, May 2, 2013 at 3:51 PM, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Mon, Apr 29, 2013 at 10:47 AM,
On Mon, May 13, 2013 at 2:21 PM, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Thu, May 9, 2013 at 2:20 PM, Sriraman Tallam tmsri...@google.com wrote:
cc:Diego
On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Thu, May 2, 2013 at 3:51 PM, Sriraman
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,
Ping.
On Thu, May 9, 2013 at 2:22 PM, Sriraman Tallam tmsri...@google.com wrote:
cc:Diego
On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Thu, Apr 25, 2013 at 4:50 PM, Sriraman Tallam tmsri...@google.com wrote:
Attaching an updated patch.
Thanks
Sri
I still have not heard from i386 or ia64 folks. Anyone?
On 4/24/13, Lawrence Crowl cr...@googlers.com wrote:
This patch is a consolodation of the hash_table patches to the
cxx-conversion branch for files under gcc/config.
Recipients:
config/arm/arm.c - ni...@redhat.com,
run-time, according to my reading of
http://gcc.gnu.org/codingconventions.html .
I chenged it, but I note that runtime is used much more often than
run-time and run time in the docs.
However, runtime is preferred for libraries and system support
present at run time, which is what this option
Hi Jakub. Hi folks.
Attached is a patch against the gomp4 branch implementing Cilk's
#pragma simd construct with the gomp4 infrastructure. I emit OMP_SIMD
as previously mentioned, and let omp-low.c do the rest.
The reason for this, is that Cilk Plus and OMP4 behave pretty much the
same
On May 9, 2013, Jason Merrill ja...@redhat.com wrote:
On 05/01/2013 07:40 PM, Mike Stump wrote:
$ bash -c trap 'echo remove lock' 0; true; echo $?
Thanks for the suggestion. Here's a revised patch:
I like the idea of the patch, for I feel your pain too ;-)
However, rather than
Hi,
I would prefer to encode section changes within a function in the
debug info. I will explain later today.
Ciao!
Steven
On 5/13/13, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Thu, May 9, 2013 at 2:22 PM, Sriraman Tallam tmsri...@google.com
wrote:
cc:Diego
On Tue, May 7,
69 matches
Mail list logo
