On Mon, May 13, 2013 at 11:46 PM, H.J. Lu wrote:
> On Mon, Apr 29, 2013 at 10:47 AM, Sriraman Tallam
> wrote:
>> On Thu, Apr 25, 2013 at 12:41 PM, Joseph S. Myers
>> wrote:
>>> On Tue, 16 Apr 2013, Sriraman Tallam wrote:
>>>
Ok, it is on by default now. There
Hi!
This patch adds safelen field to struct loop, teaches expand_omp_simd
to set it on the simd loops and then uses it in a few places:
1) because the loops are explicitly marked for vectorization by the user,
we'll try to ifconvert them and vectorize even without -O3, -Ofast or
-ftree-vecto
On 05/13/13 14:09, Jan Hubicka wrote:
Index: varasm.c
===
I think DECL_COMDAT is not what you really want to return true for. So perhaps
you really want (TREE_PUBLIC (decl) && decl_binds_to_current_def_p)?
Like this? This too
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 wrote:
> Ping.
>
> On Thu, May 9, 2013 at 2:22 PM, Sriraman Tallam
> wrote:
>> cc:Diego
>>
>> On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam
On May 9, 2013, Jason Merrill 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 implementing the locki
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 w
> "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
I still have not heard from i386 or ia64 folks. Anyone?
On 4/24/13, Lawrence Crowl 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, ramana.radhakrish...@arm.com
> con
Ping.
On Thu, May 9, 2013 at 2:22 PM, Sriraman Tallam wrote:
> cc:Diego
>
> On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam wrote:
>> Ping.
>>
>> On Thu, Apr 25, 2013 at 4:50 PM, Sriraman Tallam wrote:
>>> Attaching an updated patch.
>>>
>>> Thanks
>>> Sri
>>>
>>> On Thu, Apr 25, 2013 at 4:42 P
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
* libsupc++/Makefile.am (sources): Add bad_array_length.cc,
bad_array_new.cc.
On Mon, May 13, 2013 at 2:21 PM, Sriraman Tallam wrote:
> Ping.
>
> On Thu, May 9, 2013 at 2:20 PM, Sriraman Tallam wrote:
>> cc:Diego
>>
>> On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam wrote:
>>> Ping.
>>>
>>> On Thu, May 2, 2013 at 3:51 PM, Sriraman Tallam wrote:
Ping.
On Mo
Ping.
On Thu, May 9, 2013 at 2:20 PM, Sriraman Tallam wrote:
> cc:Diego
>
> On Tue, May 7, 2013 at 2:41 PM, Sriraman Tallam wrote:
>> Ping.
>>
>> On Thu, May 2, 2013 at 3:51 PM, Sriraman Tallam wrote:
>>> Ping.
>>>
>>> On Mon, Apr 29, 2013 at 10:47 AM, Sriraman Tallam
>>> wrote:
On Thu,
On 05/13/2013 09:39 AM, Jakub Jelinek wrote:
> 2013-05-13 Jakub Jelinek
>
> PR middle-end/57251
> * expr.c (expand_expr_real_2) : Handle
> the case when both op0 and op1 have VOIDmode.
>
> * gcc.dg/torture/pr57251.c: New test.
Ok.
r~
On Tue, May 14, 2013 at 12:45 AM, Tom Tromey wrote:
>> "Alexander" == Alexander Monakov 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 th
> "Alexander" == Alexander Monakov 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 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
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.
Becaus
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
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 we
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 att
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
Date: Mon May 13 13:26:12 2013 -0400
PR c++/57041
* decl.c (reshape_init_class): Handle error_mark
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
Date: Mon May 13 10:45:41 2013 -0400
PR c++/57252
* decl.c (decls_match): Compare ref-qualifie
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;
13526:
OK for google branches.
On Thu, May 9, 2013 at 1:40 PM, Han Shen(沈涵) 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.
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 s
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
* compare_tests: Limit lines to 2000 characters as comm on Mac OS
X 10.8.3 doesn't like long lines (those 2055 characters or more).
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
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
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
> 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
>
> * config/rl78/rl78.md (mulsi3_g13): Add additional 'nop' required
> in multiply-accumulate mode
> 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
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
On Mon, May 13, 2013 at 6:43 PM, Jakub Jelinek 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
>> add an
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, 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 {n
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 sin
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.
T
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
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, p
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. Ho
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
re-sending the patch
On Mon, May 13, 2013 at 5:54 PM, Evgeniy Stepanov wrote:
> friendly ping
>
> On Mon, Apr 22, 2013 at 5:23 PM, Evgeniy Stepanov wrote:
>> Hi,
>>
>> this patch disables rdrand in c++11/random.cc when building with Clang
>> compiler. Current Clang misses a number of definitions
2013/5/13 Steven Bosscher :
> 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
>>> +msp430
OK
/Marcus
On 13 May 2013 11:38, Sofiane Naci 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
friendly ping
On Mon, Apr 22, 2013 at 5:23 PM, Evgeniy Stepanov 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 google/integ
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, skip_consecuti
> 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
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
On Mon, May 13, 2013 at 7:25 AM, Florian Weimer 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 its sizeof,
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
* c-tree.h (c_finish_omp_declare_simd): New prototype.
* c-typeck.c (c_
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 RTL
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 thr
On Mon, May 13, 2013 at 1:54 PM, Gopalasubramanian, Ganesh
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
processors in processor_
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, 13 May 2013, Richard Biener wrote:
On Sat, May 11, 2013 at 11:38 AM, Marc Glisse 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: callin
Rainer Orth 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
> Pascal Obry
>
> * gcc-interface/Makefile.in (targ): Fix target name check.
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 a
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 this
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
On Sat, May 11, 2013 at 11:38 AM, Marc Glisse 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 and build1 if
> 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 se
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 predicate
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 7,
On Sat, May 11, 2013 at 1:39 PM, Jan Hubicka 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 m
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 che
On Mon, May 13, 2013 at 5:45 AM, Kugan
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 that RHS expression value can
On Sun, May 12, 2013 at 2:04 PM, Marc Glisse 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 require
> some he
On Sat, May 11, 2013 at 1:41 AM, DJ Delorie 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, the
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:22 AM, Steven Bosscher 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 insns reading read
On Fri, May 10, 2013 at 8:27 PM, Carrot Wei 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 TREE_STATIC
> whe
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 Jeline
72 matches
Mail list logo
