Il 10/04/2013 21:33, Steven Bosscher ha scritto:
> Hello,
>
> df_find_def and df_find_use do not work properly for hard registers
> because rtx_equal_p returns false for the case where
> REGNO(x)==REGNO(y) but the modes are different. This happened as
> follows in my case:
>
> Breakpoint 9, df_re
On Wed, Apr 10, 2013 at 09:04:06PM -0500, Gabriel Dos Reis wrote:
> We might be saying the same thing using different languages.
>
> I was the %r/%R markers are ways of implementing the IL language
> I suggested in that message. So, as such I do not object to it.
> Having an explicit call makes t
On Wed, Apr 10, 2013 at 05:16:17PM -0700, Andrew Pinski wrote:
> On Wed, Apr 10, 2013 at 3:24 PM, Aldy Hernandez wrote:
> > Hi Tom. Hi folks.
> >
> > We've asked Balaji to rewrite the <#pragma simd> handling for cilkplus as we
> > currently do for OMP, etc, in init_pragma().
> >
> > The cilkplus
> -Original Message-
> From: Vladimir Makarov [mailto:vmaka...@redhat.com]
> Sent: Thursday, April 11, 2013 7:20 AM
> To: Bin Cheng
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH GCC/pr56124] Don't prefer memory if the source of load
> operation has side effect
>
> On 13-04-06 11:16
Applied, thanks.
Jason
OK.
Jason
On Wed, Apr 10, 2013 at 12:54 PM, Manuel López-Ibáñez
wrote:
> On 8 April 2013 21:06, Jakub Jelinek wrote:
>> On Mon, Apr 08, 2013 at 07:54:18PM +0200, Manuel López-Ibáñez wrote:
>>> > can be right now a single call, while you would need several. Also, if
>>> > you
>>> > eventually want to colo
On Wed, Apr 10, 2013 at 1:42 PM, Manuel López-Ibáñez
wrote:
> On 9 April 2013 15:21, Jakub Jelinek wrote:
>> white). The default is still -fdiagnostics-color=never, can be changed
>> later on.
>
> Apart from my comments elsewhere
> (http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00614.html), the p
On 10 April 2013 18:48, Marcus Shawcroft wrote:
> Zhenqiang, Does Jame's patch fix your test case?
Thank you all. The patch fixes my test case.
-Zhenqiang
> On 10 April 2013 11:43, Richard Earnshaw wrote:
>> On 10/04/13 11:31, James Greenhalgh wrote:
>>>
>>>
-Original Message-
>>>
On Wed, Apr 10, 2013 at 3:24 PM, Aldy Hernandez wrote:
> Hi Tom. Hi folks.
>
> We've asked Balaji to rewrite the <#pragma simd> handling for cilkplus as we
> currently do for OMP, etc, in init_pragma().
>
> The cilkplus branch currently has something like:
>
> cpp_register_deferred_pragma (
Hi,
This patch
1. Uses relative line offset (lineno - start_lineno_of_function) to
represent AutoFDO profile. This ensures profile still work for
modified source code.
2. When matching the profile, add function name (bfd_name) to match
the inline stack.
Bootstrapped and passed regression tests.
On 13-04-06 11:16 PM, Bin Cheng wrote:
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On
Behalf Of Bin Cheng
Sent: Monday, March 25, 2013 3:15 PM
To: gcc-patches@gcc.gnu.org
Subject: FW: [PATCH GCC/pr56124] Don't prefer memory if the sour
On Wed, Apr 10, 2013 at 09:18:46AM -0700, Mike Stump wrote:
> On Apr 9, 2013, at 11:33 AM, Janne Blomqvist
> wrote:
> > the attached patch reduces the size of module files on disk
>
> Do those modules interoperate with C++ modules flawlessly? :-)
Fortran 2008 became an ISO well before C++'s l
Hi Tom. Hi folks.
We've asked Balaji to rewrite the <#pragma simd> handling for cilkplus
as we currently do for OMP, etc, in init_pragma().
The cilkplus branch currently has something like:
cpp_register_deferred_pragma (parse_in, "simd", "",
PRAGMA_SIM
I took the opportunity to merge the trunk into the branch (Rev. 197683)
after Janne's touch-all-Fortran-file bool patch.
Tobias
Hi!
This patch
1) starts using the new GOMP_parallel* (without _start) APIs for #pragma omp
parallel
2) makes proc_bind clauses work
3) handles #pragma omp taskgroup (though, on the library side it is just a
dummy right now)
Tested on x86_64-linux, committed to gomp-4_0-branch.
2013-04-10 J
OK. Thanks.
/Marcus
On 10 April 2013 11:32, Hurugalawadi, Naveen
wrote:
> Hi,
>
> Please find attached the patch that implements negs instruction
> with shift for aarch64 target.
> Testcase have been added for negs instruction.
>
> Please review the same and let me know if there should be any
> m
Fortran 2008 supports C_LOC(array); if the argument is not simply
contiguous, the current code adds a call to __gfortran_intrinsic_pack.
The pack call shouldn't be there. Fortran 2008 demands that the actual
argument is contiguous and intrinsic_pack copy creates a copy if the
run-time check s
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56903
In this test case reload pass gets correct value HARD_REGNO_MODE_OK
because it can not create pseudos (can_create_pseudo) and this was
actually used that we are in reload (or after reload). LRA can create
pseudos ther
From: David Miller
Date: Mon, 08 Apr 2013 21:56:04 -0400 (EDT)
>
> One major suboptimal area of the sparc back end is cstore generation
> on 64-bit.
>
> Due to the way arguments and return values of functions must be
> promoted, the ideal mode for cstore's result would be DImode.
>
> But this
Hello,
df_find_def and df_find_use do not work properly for hard registers
because rtx_equal_p returns false for the case where
REGNO(x)==REGNO(y) but the modes are different. This happened as
follows in my case:
Breakpoint 9, df_reg_used (insn=0x3fffb6083c50, reg=0x3fffb5eb72e0) at
../../trunk/g
This patch:
http://gcc.gnu.org/ml/gcc-patches/2009-04/msg01807.html
fixes ELF .size directive for structures with a flexible arrray member:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28865
Can someone take a look?
--
H.J.
This is yet another bug just like PR target/52610, we need to pass
-Av8 to the assembler for every cpu type that will use MASK_V8 in
sparc.c:sparc_option_override().
I found this while testing GMP builds.
I'd like to check this into the various gcc-4_X-branch branches as
well, unless there are m
On Wed, Apr 10, 2013 at 09:53:09PM +0400, Michael Zolotukhin wrote:
> > Hi, I am writing memcpy for libc. It avoids computed jump and has is
> > much faster on small strings (variant for sandy bridge attached.
>
> I'm not sure I get what you meant - could you please explain what is
> computed jump
On 9 April 2013 15:21, Jakub Jelinek wrote:
> white). The default is still -fdiagnostics-color=never, can be changed
> later on.
Apart from my comments elsewhere
(http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00614.html), the patch
looks fine to me. But perhaps we should change the default to aut
On 04/10/2013 09:31 AM, Jakub Jelinek wrote:
Hi!
As f1 in the testcase shows, applying distributive law in extract_muldiv_1
isn't safe if overflow behavior isn't defined, if we have
(op0 + c1) * c2
and the type is signed, we can't just try to fold that to
op0 * c2 + (c1 * c2)
even when we know t
On 8 April 2013 21:06, Jakub Jelinek wrote:
> On Mon, Apr 08, 2013 at 07:54:18PM +0200, Manuel López-Ibáñez wrote:
>> > can be right now a single call, while you would need several. Also, if you
>> > eventually want to colorize something in say error_at, warning_at and
>> > similar format strings
> Hi, I am writing memcpy for libc. It avoids computed jump and has is
> much faster on small strings (variant for sandy bridge attached.
I'm not sure I get what you meant - could you please explain what is
computed jumps?
> You must also check performance with cold instruction cache.
> Now memcp
On Wed, Apr 10, 2013 at 08:14:30PM +0400, Michael Zolotukhin wrote:
> Hi,
> This patch adds a new algorithm of expanding movmem in x86 and a bit
> refactor existing implementation. This is a reincarnation of the patch
> that was sent wasn't checked couple of years ago - now I reworked it
> from scr
On 10/04/13 11:35, Hurugalawadi, Naveen wrote:
+(define_insn "*cmn_swp__reg"
+ [(set (reg:CC_SWP CC_REGNUM)
+ (compare:CC_SWP (ANY_EXTEND:GPI
+(match_operand:ALLX 0 "register_operand" "r"))
+ (neg:GPI (match_operand:GPI 1 "register_operand"
"
http://gcc.gnu.org/gcc-4.9/changes.html was so empty :-)
I have committed the attached patch - comments and suggestions are welcome.
Tobias
Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.9/changes.html,v
retrieving
Hello!
Attached testsuite patch fixes:
pr33992.c: In function ‘do_test’:
pr33992.c:11:1: error: inlining failed in call to always_inline ‘foo’:
function body can be overwritten at link time
pr33992.c:28:9: error: called from here
foo (r);
^
errors through gcc and g++ testsuite when
On Wed, Apr 10, 2013 at 06:42:58PM +0200, Marek Polacek wrote:
> On Wed, Apr 10, 2013 at 11:49:18AM +0200, Jakub Jelinek wrote:
> > Shouldn't this be again solved instead by bumping minimum for the param to 1
> > from 0? Because, the smaller the param is, the bigger freq_threshold is,
> > so if fo
This patch integrated Martin's comments.
-Rong
2013-04-10 Rong Xu
* contrib/profile_tool: New
* gcc/Makefile.in (GCC_INSTALL_NAME): install profile_tool
Index: contrib/profile_tool
===
--- contrib/profile_tool
On Wed, Apr 10, 2013 at 11:49:18AM +0200, Jakub Jelinek wrote:
> Shouldn't this be again solved instead by bumping minimum for the param to 1
> from 0? Because, the smaller the param is, the bigger freq_threshold is,
> so if for the smallest param we suddenly set freq_threshold to 0, it isn't
> co
Janus Weil wrote:
Ok, here is an updated patch, which does the discussed checking for
procedure pointer assignments. For this I have introduced a new
function 'gfc_explicit_interface_required', which checks all the items
in F08:12.4.2.2 and is loosely based on the present checks in
'resolve_globa
On Apr 9, 2013, at 11:33 AM, Janne Blomqvist wrote:
> the attached patch reduces the size of module files on disk
Do those modules interoperate with C++ modules flawlessly? :-)
Hi,
This patch adds a new algorithm of expanding movmem in x86 and a bit
refactor existing implementation. This is a reincarnation of the patch
that was sent wasn't checked couple of years ago - now I reworked it
from scratch and divide into several more manageable parts.
For now this algorithm is
On 04/10/2013 12:02 PM, Mike Stump wrote:
On Apr 10, 2013, at 12:38 AM, Richard Biener wrote:
Yeah, I think we want to test ~(T)0<(T)0 here.
Thanks Lawrence, in the next version of the patch, you will discover this at
the bottom if you look hard. :-)
actually closer to the middle.
On Apr 10, 2013, at 12:38 AM, Richard Biener wrote:
> Yeah, I think we want to test ~(T)0<(T)0 here.
Thanks Lawrence, in the next version of the patch, you will discover this at
the bottom if you look hard. :-)
Am 10.04.2013 16:21, schrieb Janus Weil:
2013/4/7 Tobias Burnus :
Thus, the only place where the check can be is for:
f => ff
In your example, the explicit interface of "ff" is known thus it should
be
testable at resolution time of the proc-pointer assignment.
Right. However, strictly spe
Hi!
As f1 in the testcase shows, applying distributive law in extract_muldiv_1
isn't safe if overflow behavior isn't defined, if we have
(op0 + c1) * c2
and the type is signed, we can't just try to fold that to
op0 * c2 + (c1 * c2)
even when we know that c1*c2 doesn't overflow, because op0 * c2
mi
On 10/04/13 15:44, Yufeng Zhang wrote:
Hi,
This patch changes the compiler to correctly generate .arch and .cpu
assembly directives in order to support the inline assembly of
instructions that are part of a feature, e.g. crypto.
OK for the trunk?
Thanks,
Yufeng
gcc/
* config/aarch
Hi,
This patch changes the compiler to correctly generate .arch and .cpu
assembly directives in order to support the inline assembly of
instructions that are part of a feature, e.g. crypto.
OK for the trunk?
Thanks,
Yufeng
gcc/
* config/aarch64/aarch64.c (aarch64_print_extension):
* PING *
Tobias Burnus:
An unallocated polymorphic variable has the declared type; however,
for static (SAVE) variables, the current code didn't set the value.
(That the end of scope deallocation/_gfortran_caf_deregister is gone
for coarrays (declared in the main program) was a side effect. T
2013/4/7 Tobias Burnus :
>>> Thus, the only place where the check can be is for:
>>>
>>>f => ff
>>>
>>> In your example, the explicit interface of "ff" is known thus it should
>>> be
>>> testable at resolution time of the proc-pointer assignment.
>>
>> Right. However, strictly speaking, the poi
Many compilers have some pragma or directive to disable the type, kind
and rank (TKR) checks. That feature matches C's "void*" pointer and can
be used in conjunction with passing some byte data to a procedure, which
only needs to know either the pointer address or pointer address and size.
I t
This handles commutative operations during SLP tree build in the
way that if one configuration does not match, the build will
try again with commutated operands for. This allows to remove
the special-casing of commutated loads in a complex addition
that was in the end handled as "permutation". I
Hi,
This patch adds a -gdwarf option to make gcc generate DWARF with the
default version (currently 4).
A bunch of tests in the dejagnu dwarf test suite fail right now if gcc
is configured/built with support for multiple debugging formats and
DWARF is not the default. The failing tests don't expl
Ping
2013/4/3 Maksim Kuznetsov :
> Thank you for your feedback!
>
>> For '}' case, can you simply just add
>>
>> /* Skip over any character after a percent sign. */
>> if (*p == '%' && *(p + 1))
>> {
>> p += 2;
>> continue;
>> }
>>
>> without changing the do-while loop to the while loop?
>
>
Hi,
this issue is about some enumeration types which are strictly speaking
illegal and we are accepting nonetheless:
enum {}; //-std=c++98 or -std=c++11
enum class {}; //-std=c++11
enum class { x }; //-std=c++11
I suppose we want to be less strict about the former thus I'm using a
pedwarn
On Thu, 21 Mar 2013, Richard Biener wrote:
> On Tue, 19 Mar 2013, Richard Biener wrote:
>
> > On Tue, 19 Mar 2013, Richard Biener wrote:
> >
> > > On Tue, 19 Mar 2013, Richard Biener wrote:
> > >
> > > >
> > > > This adds a GC collection point after each pass instead just after
> > > > those w
On Wed, Apr 10, 2013 at 12:50 PM, James Greenhalgh
wrote:
>
>> -Original Message-
>> From: dosr...@gmail.com [mailto:dosr...@gmail.com] On Behalf Of Gabriel
>> Dos Reis
>> Sent: 20 March 2013 19:09
>> To: James Greenhalgh
>> Cc: Jakub Jelinek; Richard Biener; gcc-patches@gcc.gnu.org; Jason
Hi!
I've committed the following set of changes to gomp-4_0-branch after
regtesting. This adds (so far dummy) exports for the new OpenMP 4.0
library functions, and planned entry points for #pragma omp cancel*,
plus, as discussed with Richard privately also new GOMP_parallel* entry
points, because
> -Original Message-
> From: dosr...@gmail.com [mailto:dosr...@gmail.com] On Behalf Of Gabriel
> Dos Reis
> Sent: 20 March 2013 19:09
> To: James Greenhalgh
> Cc: Jakub Jelinek; Richard Biener; gcc-patches@gcc.gnu.org; Jason
> Merrill; m...@codesourcery.com
> Subject: Re: [C++11][4.9] Add
Zhenqiang, Does Jame's patch fix your test case?
/Marcus
On 10 April 2013 11:43, Richard Earnshaw wrote:
> On 10/04/13 11:31, James Greenhalgh wrote:
>>
>>
>>> -Original Message-
>>> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
>>> ow...@gcc.gnu.org] On Behalf Of Zhenqiang Ch
On 10/04/13 11:31, James Greenhalgh wrote:
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen
Sent: 10 April 2013 09:02
To: gcc-patches@gcc.gnu.org
Cc: Marcus Shawcroft
Subject: [PATCH, AARCH64] Fix unrecognizable i
Hi,
Please find attached the patch that implements compare negative
instruction with shift and extend mode for aarch64 target.
Testcase have been added for compare and compare negative instruction.
Please review the same and let me know if there should be any
modifications in the patch.
Build
Hi,
Please find attached the patch that implements negs instruction
with shift for aarch64 target.
Testcase have been added for negs instruction.
Please review the same and let me know if there should be any
modifications in the patch.
Build and tested on aarch64-thunder-elf (using Cavium's in
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen
> Sent: 10 April 2013 09:02
> To: gcc-patches@gcc.gnu.org
> Cc: Marcus Shawcroft
> Subject: [PATCH, AARCH64] Fix unrecognizable insn issue
>
> Hi,
>
> During exp
Thanks Richard, nice adding, regression passed on my x86_64 GNU/Linux
On Wed, Apr 10, 2013 at 5:22 PM, Richard Biener
wrote:
> On Tue, Apr 9, 2013 at 5:10 PM, Zhouyi Zhou wrote:
>> Hi Richard,
>> I do not have write access to GCC SVN repository, can you commit it for me?
>> Thanks alot
>
> Commi
On Wed, Apr 10, 2013 at 11:44:18AM +0200, Marek Polacek wrote:
> Ping.
>
> On Thu, Apr 04, 2013 at 04:01:09PM +0200, Marek Polacek wrote:
> > This fixes an ICE in case we use weirdo --param value for
> > PARAM_ALIGN_THRESHOLD. The patch is by Andrew; I added CL, testcase
> > and tested.
> >
> >
On 10 April 2013 17:18, Andrew Pinski wrote:
> On Wed, Apr 10, 2013 at 2:02 AM, Zhenqiang Chen
> wrote:
>> On 10 April 2013 16:05, Andrew Pinski wrote:
>>> On Wed, Apr 10, 2013 at 1:02 AM, Zhenqiang Chen
>>> wrote:
Hi,
During expand, function aarch64_vcond_internal inverses some
Ping.
On Thu, Apr 04, 2013 at 04:01:09PM +0200, Marek Polacek wrote:
> This fixes an ICE in case we use weirdo --param value for
> PARAM_ALIGN_THRESHOLD. The patch is by Andrew; I added CL, testcase
> and tested.
>
> Bootstrapped/regtested on x86_64-linux, ok for trunk? And what about other
>
On Tue, Apr 9, 2013 at 5:10 PM, Zhouyi Zhou wrote:
> Hi Richard,
> I do not have write access to GCC SVN repository, can you commit it for me?
> Thanks alot
Committed with adding { dg-require-weak "" } to the testcase.
Richard.
> Cheers
> Zhouyi
>
>
> On Tue, Apr 9, 2013 at 5:04 PM, Richard Bie
On Wed, Apr 10, 2013 at 2:02 AM, Zhenqiang Chen
wrote:
> On 10 April 2013 16:05, Andrew Pinski wrote:
>> On Wed, Apr 10, 2013 at 1:02 AM, Zhenqiang Chen
>> wrote:
>>> Hi,
>>>
>>> During expand, function aarch64_vcond_internal inverses some CMP, e.g.
>>>
>>> a LE b -> b GE a
>>>
>>> But if "b"
2013/4/9 Eric Botcazou :
>> On behalf of Andes Technology Co., we have signed FSF agreement.
>> However, so far I don't have svn write access yet.
>> Would you please help to commit this patch?
>
> Jeff has kindly offered to sponsor you for write access, so you should be able
> to install it yourse
This makes us handle mixed constant / invariant ops in SLP
(constant ops are a subset of invariant ops). This allows us
to vectorize both foo and bar (the vect_get_constant_vectors
adjustment is only necessary for foo).
The patch also simplifies type checking (which in the end I think
is not req
On 10 April 2013 16:05, Andrew Pinski wrote:
> On Wed, Apr 10, 2013 at 1:02 AM, Zhenqiang Chen
> wrote:
>> Hi,
>>
>> During expand, function aarch64_vcond_internal inverses some CMP, e.g.
>>
>> a LE b -> b GE a
>>
>> But if "b" is "CONST0_RTX", "b GE a" will be an illegal insn.
>>
>> Refer http
On Tue, Apr 9, 2013 at 6:24 PM, Joern Rennecke
wrote:
> Quoting Richard Biener :
>
>> I don't see that. It's merely a complication of optimal handling of
>> a * b +- c * d vs. just a * b +- c. The pass does simple pattern matching
>> only, not doing a global optimal transform, so adding another
This splits out load permutation computation from SLP tree building.
It also removes the broken support for swapping mismatched operands.
If it ever triggers we'll ICE later because:
case vect_internal_def:
! if (different_types)
! {
! oprnd0_info = (*opr
On Wed, Apr 10, 2013 at 1:02 AM, Zhenqiang Chen
wrote:
> Hi,
>
> During expand, function aarch64_vcond_internal inverses some CMP, e.g.
>
> a LE b -> b GE a
>
> But if "b" is "CONST0_RTX", "b GE a" will be an illegal insn.
>
> Refer https://bugs.launchpad.net/linaro-toolchain-binaries/+bug/11639
Hi,
During expand, function aarch64_vcond_internal inverses some CMP, e.g.
a LE b -> b GE a
But if "b" is "CONST0_RTX", "b GE a" will be an illegal insn.
Refer https://bugs.launchpad.net/linaro-toolchain-binaries/+bug/1163942
for detail about the issue.
The patch is to make "b" a register wh
On Tue, Apr 9, 2013 at 9:15 PM, Marc Glisse wrote:
> On Tue, 9 Apr 2013, Marc Glisse wrote:
>
>> On Tue, 9 Apr 2013, Richard Biener wrote:
>>
>>> I seem to remember discussion in the PR(s) that the intrinsics should
>>> (and do for other compilers) expand to the desired instructions even when
>>>
On Tue, Apr 9, 2013 at 5:36 PM, Lawrence Crowl wrote:
>
> On Apr 9, 2013 2:02 AM, "Richard Biener" wrote:
>>
>> On Mon, Apr 8, 2013 at 10:39 PM, Lawrence Crowl wrote:
>> > On 4/8/13, Kenneth Zadeck wrote:
>> >> The other problem, which i invite you to use the full power of
>> >> your c++ sorcer
Adding myself to the list of members in "Write After Approval" section.
Index: ChangeLog
===
--- ChangeLog (revision 197662)
+++ ChangeLog (working copy)
@@ -1,3 +1,7 @@
+2013-04-10 Chung-Ju Wu
+
+ * MAINTAINERS (Write A
On Wed, Apr 10, 2013 at 12:17:31AM +0200, Tobias Burnus wrote:
> +/* Output debug information for namelists. */
> +
> +void
> +dwarf2out_namelist_decl (const char *name, tree context,
> + vec *item_decls)
> +{
> + dw_die_ref scope_die, nml_die, nml_item_die, nml_item_ref_die
77 matches
Mail list logo
