Re: Make m32c build, fix PSImode truncation

2013-04-30 Thread Richard Sandiford
DJ Delorie writes: >> Sorry for missing the truncation patterns, I should have grepped >> more than m32c.md. They look a lot like normal moves though. Is >> truncation really not a noop, or are the patterns there to work >> around something (probably this :-))? > > Not sure which pattern you're

Re: [PATCH] Don't instrument with -fsanitize=thread accesses to DECL_HARD_REGISTER vars (PR tree-optimization/57104)

2013-04-30 Thread Richard Biener
On Mon, Apr 29, 2013 at 9:14 PM, Jakub Jelinek wrote: > Hi! > > DECL_HARD_REGISTER vars don't live in memory, thus they can't be > addressable. > > The following patch fixes the ICE, ok for trunk/4.8? Ok. Thanks, Richard. > 2013-04-29 Jakub Jelinek > > PR tree-optimization/57104 >

Re: [Fortran-Dev] Some ubounds -> extent changes

2013-04-30 Thread Tobias Burnus
Tobias Burnus wrote: This patch changes some ubounds to extent. The patch is relative to my type patch - but it also applies without. It also fixes a bunch fo testsuite failures. Build and regtested on x86-64-gnu-linux. I have now committed it to the branch as Rev. 198440. Additionally, I hav

[PATCH, wwwdocs] Fix PR 50642

2013-04-30 Thread Shakthi Kannan
Hi, The following patch overrides the default @smallexample attributes defined by makeinfo. It includes a html.css file where the smallexample attributes are defined to use a medium font size with a light blue background for readability and clarity. 2013-04-30 Shakthi Kannan PR wwwdocs/5

Re: [Fortran-dev] Implement TS29113 type handling

2013-04-30 Thread Tobias Burnus
Tobias Burnus wrote: This patch implements the TS29113 type handling. I have now committed this patch as Rev. 198447, http://gcc.gnu.org/ml/gcc-cvs/2013-04/msg01155.html The commit contains the -m32 corrections of Dominique, http://gcc.gnu.org/ml/fortran/2013-04/msg00248.html Tobias

Re: [PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS

2013-04-30 Thread Matthew Gretton-Dann
Hi, On 08/04/13 06:28, Xinyu Qi wrote: At 2013-04-02 17:50:03,"Ramana Radhakrishnan" wrote: On 04/02/13 10:40, Xinyu Qi wrote: Hi, According to Vladimir Makarov's analysis, the root cause of PR target/54338 is that ALL_REGS doesn't contain IWMMXT_GR_REGS in REG_CLASS_CONTENTS. It see

[PATCH] Fix PRs 56957 and 57105

2013-04-30 Thread Andrey Belevantsev
Hello, After Steven's changes tightening the add/remove insn interface, we found a problem in the selective scheduling when we incorrectly determined whether an insn should be removed (and a fresh one emitted) or just moved to a scheduling point. Fixed by just comparing INSN_UIDs of the insn

Re: [patch, mips] Fix for PR target/56942

2013-04-30 Thread Steven Bosscher
(Top post is gmail's fault ;-) Hello, I dont like this at all. At the very least, if we go this way, then all places where next_active_insn is used should be updated. Otherwise this is just confusion proliferation. Before my patch most ports used the "active" variants and I specifically did

Ping: [PATCH, PR 42371] Remove references to functions from symbol table during inlining

2013-04-30 Thread Martin Jambor
Ping. Thanks, Martin - Forwarded message from Martin Jambor - Date: Wed, 17 Apr 2013 17:45:17 +0200 From: Martin Jambor To: GCC Patches Cc: Jan Hubicka Subject: [PATCH, PR 42371] Remove references to functions from symbol table during inlining Hi, the patch below is a fix for PR

Ping: [PATCH, PR 56988] Honor by_ref in IPA-CP transformation phase

2013-04-30 Thread Martin Jambor
Ping. Thanks, Martin - Forwarded message from Martin Jambor - Date: Fri, 19 Apr 2013 11:05:58 +0200 From: Martin Jambor To: GCC Patches Cc: Jan Hubicka Subject: [PATCH, PR 56988] Honor by_ref in IPA-CP transformation phase Hi, the following patch fixes PR 56988 by distinguishing b

Re: [C++ Patch/RFC] PR 57092

2013-04-30 Thread Paolo Carlini
Hi, On 04/29/2013 04:48 PM, Jason Merrill wrote: On 04/29/2013 05:05 AM, Paolo Carlini wrote: in this 4.8/4.9 Regression, finish_decltype_type doesn't handle ADDR_EXPR. Hmm...we're seeing the regression because previously finish_decltype_type would have just returned the type of the template

[PATCH] Fix PR57122, bug in split_edge

2013-04-30 Thread Richard Biener
The following fixes PR57122 where splitting an edge from the loop latch is wrongly detected as latch-edge split (happens in case an irreducible region is inside the loop). Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 2013-04-30 Richard Biener PR middle-end/571

Re: Ping: [PATCH, PR 56988] Honor by_ref in IPA-CP transformation phase

2013-04-30 Thread Jan Hubicka
> 2013-04-18 Martin Jambor > > PR middle-end/56988 > * ipa-prop.h (ipa_agg_replacement_value): New flag by_ref. > * ipa-cp.c (ipa_get_indirect_edge_target_1): Also check that by_ref > flags match. > (find_aggregate_values_for_callers_subset): Fill in the by_ref fla

Re: [C++ Patch/RFC] PR 57092

2013-04-30 Thread Jason Merrill
On 04/30/2013 08:03 AM, Paolo Carlini wrote: I'm wondering if handling the additional TREE_CODE in finish_decltype_type isn't overall preferable (assuming we wouldn't end up soon handling all sorts of *_EXPR ;) That's exactly the problem; it wouldn't stop with ADDR_EXPR, we would need to handl

Re: [patch, mips] Fix for PR target/56942

2013-04-30 Thread Richard Sandiford
Steven Bosscher writes: > I dont like this at all. At the very least, if we go this way, > then all places where next_active_insn is used should be updated. > Otherwise this is just confusion proliferation. You mean all places where next_active_insn is used to get the jump table? That would

[PATCH] Fix PR57107

2013-04-30 Thread Richard Biener
This fixes PR57107, a latent bug in clobber sinking. We cannot do without keeping virtual SSA form up-to-date when sinking clobbers through multiple forwarders. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk (latent on the branch I think). Richard. 2013-04-30 Richard Bi

Re: [PATCH, wwwdocs] Fix PR 50642

2013-04-30 Thread Joseph S. Myers
On Tue, 30 Apr 2013, Shakthi Kannan wrote: > The following patch overrides the default @smallexample attributes > defined by makeinfo. It includes a html.css file where the smallexample > attributes are defined to use a medium font size with a light blue > background for readability and clarity

Re: [build, driver] RFC: Support compressed debug sections

2013-04-30 Thread Rainer Orth
"Joseph S. Myers" writes: > On Thu, 11 Apr 2013, Rainer Orth wrote: > >> +gz= >> +Common Driver JoinedOrMissing >> +-gz=Generate compressed debug sections in format > > Although handled entirely in specs, I think it's best to use the Enum .opt > facility to list the valid arguments to t

Re: [patch] Fix node weight updates during ipa-cp (issue7812053)

2013-04-30 Thread Teresa Johnson
On Mon, Apr 29, 2013 at 12:35 PM, H.J. Lu wrote: > On Mon, Apr 29, 2013 at 10:31 AM, Teresa Johnson wrote: >> FYI, Fixed in r198416. >> >> Thanks, >> Teresa >> > > I noticed that sometimes GCC generates: > > _8 = memcpy (ret_6, s_2(D), len_4); > _8 = memcpy (ret_6, s_2(D), len_4); > memcpy (_17,

Re: [patch, mips] Fix for PR target/56942

2013-04-30 Thread Richard Sandiford
Richard Sandiford writes: > Steven Bosscher writes: >> I dont like this at all. At the very least, if we go this way, >> then all places where next_active_insn is used should be updated. >> Otherwise this is just confusion proliferation. > > You mean all places where next_active_insn is used

Re: MEM_REF representation problem, and folding fix

2013-04-30 Thread Richard Biener
On Mon, Apr 29, 2013 at 11:20 PM, Bernd Schmidt wrote: > Currently, MEM_REF contains two pointer arguments, one which is supposed > to be a base object and another which is supposed to be a constant > offset. This representation is somewhat problematic, as not all machines > treat pointer values a

[PATCH] Convert profile scaling computations to rounding divides (issue9050044)

2013-04-30 Thread Teresa Johnson
Follow-on patch to r197595 to complete the replacement of truncating divides in profile scaling code with rounding divide equivalents using helper routines in basic-block.h. See http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00321.html for background. In addition to bootstrap and profiledbootstrap b

Re: [WIP RFH] #pragma omp declare simd (aka OpenMP elemental functions) parsing

2013-04-30 Thread Jason Merrill
On 04/29/2013 02:32 PM, Jakub Jelinek wrote: Should I copy the parser->omp_declare_simd_clauses vector pointer say into cp_declarator structure so that grokfndecl could grab it from there? That sounds good. Also, for the attributes I wonder if it wouldn't be better to finally replace the PARM

[PATCH, fixincludes] Fix NULL on AIX

2013-04-30 Thread David Edelsohn
AIX headers do not have a complete definition of NULL and the GCC build complains about "missing sentinel". The appended patch substitutes a more complete definition of NULL that satisfies GCC/G++. I also updated the void_null fix, which was discussed in 2011 to substitute a more complete definit

[Patch, Fortran] Permit allocatable/pointer attributes with BIND(C)

2013-04-30 Thread Tobias Burnus
TS29113 permits the allocatable/pointer attribute with BIND(C); this patch allows it now with -std=f2008ts. While the TS allows it also for scalars, this patch only permits it for arrays. The reason is that TS29113 requires the use of the array descriptor - and adding the support for scalars i

Re: [PATCH, wwwdocs] Fix PR 50642

2013-04-30 Thread Shakthi Kannan
Hi, - Original Message - | From: "Joseph S. Myers" | | onlinedocs copies of manuals are generated by | maintainer-scripts/update_web_docs_svn, which doesn't use any makefiles. \-- Could you please list the pre-requisite steps needed to run the above? I had used "configure; make; make h

[AArch64] Improve description of CM instructions in RTL

2013-04-30 Thread James Greenhalgh
Comparison operators can be described in RTL in a more accurate manner than simply using UNSPECs. Do this. We need the split cases for DI mode to give reload a sensible alternative. Without these, reload would churn out some truly ugly code along the lines of: > fmov d0, x0 > cmp d0, d0, #0 > fm

RFA: Use gen_int_mode in plus_constant

2013-04-30 Thread Richard Sandiford
This patch fixes out the GEN_INT/gen_int_mode that Richard pointed out in the wide-int review. It also passes "mode" rather than "VOIDmode" to immed_double_int_const. (As discussed in that thread, the latter change shouldn't make any difference in practice, but is still more correct in principle.

[AArch64] Add special case when expanding vcond with arms {-1, -1}, {0, 0}.

2013-04-30 Thread James Greenhalgh
If the end goal of a VEC_COND_EXPR is to pick between {-1, -1, -1, -1} and {0, 0, 0, 0} then we do not need to do a bit select, this is just a move of the generated mask to the result operand. This patch checks for this case, and emits the appropriate instructions. This can save us loading the tw

[AArch64 Testsuite] Fix fallout from FCM changes.

2013-04-30 Thread James Greenhalgh
Hi, gcc.target/aarch64/scalar_intrinsics.c uses the vcled_s64 style intrinsics. As these now just map to a C operation, we need to first ensure that the arguments to these functions make their way to the SIMD register set. For the >= 0 and < 0 opertaions idiom recognition will convert them to sh

[AArch64] Add combiner patterns for FAC instructions

2013-04-30 Thread James Greenhalgh
Hi, This patch adds patterns to allow the combiner to generate FAC instructions. Regression tested on aarch64-none-elf with no regressions. OK? Thanks, James --- gcc/ 2013-04-30 James Greenhalgh * config/aarch64/aarch64-simd.md (*aarch64_fac): New. * config/aarch64/iterat

[AArch64] Rewrite vca Neon patterns in C.

2013-04-30 Thread James Greenhalgh
Hi, This patch renames the vac neon intrinsics to vca as required by the newest version of the neon intrinsics spec. While we are there, we also update all the vca* intrinsics to a C implimentation. Regression tested on aarch64-none-elf with no regressions. OK? Thanks, James --- gcc/ 2013-0

[AArch64] Add testcases for FAC, FCM changes.

2013-04-30 Thread James Greenhalgh
Hi, This patch adds two testcases for the scalar and vector versions of the vca* intrinsics. New tests pass on aarch64-none-elf. OK? Thanks, James --- gcc/testsuite/ 2013-04-30 James Greenhalgh * gcc.target/aarch64/scalar-vca.c: New. * gcc.target/aarch64/vect-vca.c: Likew

[AArch64] Refactor vector max and min RTL and builtins.

2013-04-30 Thread James Greenhalgh
Hi, This patch moves around the max/min infrastructure such that we can more uniformly access the backend RTL patterns and such that it uses the standard pattern names. We also fixup the bugs this introduces in arm_neon.h which expected the older names. Regression tested on aarch64-none-elf wit

[AArch64] Fold max and min reduction builtins to tree.

2013-04-30 Thread James Greenhalgh
Hi, This patch folds the signed versions of min and max reduction builtins to tree form. We can't safely fold the unsigned versions as they do not currently have the correct type. We must use TARGET_GIMPLE_FOLD_BUILTIN as REDUC_MIN_EXPR is not a valid GENERIC code. Regression tested on aarch64-

Re: [wwwdocs] C++14 support for binary literals says Noinstead of Yes

2013-04-30 Thread Jason Merrill
I've updated the webpage. Jason

[AARCH64] Split movtf_aarch64 pattern

2013-04-30 Thread Sofiane Naci
Hi, The attached patch is part of the ongoing work to enhance instruction scheduling opportunities in AArch64. This patch splits a few alternatives of the movtf_aarch64 pattern, which used to issue 2 consecutive instructions. Tested successfully with a full aarch64-elf regression run. OK for tru

Re: [wwwdocs] C++14 support for binary literals says Noinstead of Yes

2013-04-30 Thread Jakub Jelinek
On Tue, Apr 30, 2013 at 11:23:49AM -0400, Jason Merrill wrote: > I've updated the webpage. Perhaps it would be nice if we had some testsuite coverage for it too, right now unless I'm blind there are exactly 2 testcases, using one 0b... number in each in g++.dg/. At least port gcc.dg/binary-consta

Re: [PATCH, fixincludes] Fix NULL on AIX

2013-04-30 Thread Bruce Korb
Hi David, Looks good to me, but for a small nit: On Tue, Apr 30, 2013 at 7:40 AM, David Edelsohn wrote: > +c_fix_arg = "#ifndef NULL\n" > + "#ifdef __cplusplus\n" > + "#ifdef __GNUG__\n" > + "#define NULL\t__null\n" > + "#else\t /* ! __

[AArch64] Refactor reduc_plus patterns.

2013-04-30 Thread James Greenhalgh
Hi, This patch refactors the reduc_plus RTL patterns in the AArch64 back-end. We do this to provide a more uniform interface for arm_neon.h builtins. Because of this we can rewrite the intrinsics in arm_neon.h to use these standard pattern names, and allow the signed varients to fold to tree wher

Re: Re: [wwwdocs] C++14 support for binary literals says Noinstead of Yes

2013-04-30 Thread 3dw4rd
On 04/30/13, Jakub Jelinek wrote: On Tue, Apr 30, 2013 at 11:23:49AM -0400, Jason Merrill wrote: > I've updated the webpage. Perhaps it would be nice if we had some testsuite coverage for it too, right now unless I'm blind there are exactly 2 testcases, using one 0b... number in each in g+

Re: [patch] Fix node weight updates during ipa-cp (issue7812053)

2013-04-30 Thread H.J. Lu
On Tue, Apr 30, 2013 at 7:02 AM, Teresa Johnson wrote: > On Mon, Apr 29, 2013 at 12:35 PM, H.J. Lu wrote: >> On Mon, Apr 29, 2013 at 10:31 AM, Teresa Johnson >> wrote: >>> FYI, Fixed in r198416. >>> >>> Thanks, >>> Teresa >>> >> >> I noticed that sometimes GCC generates: >> >> _8 = memcpy (ret_

Re: [C++ Patch/RFC] PR 57092

2013-04-30 Thread Paolo Carlini
On 04/30/2013 03:14 PM, Jason Merrill wrote: On 04/30/2013 08:03 AM, Paolo Carlini wrote: I'm wondering if handling the additional TREE_CODE in finish_decltype_type isn't overall preferable (assuming we wouldn't end up soon handling all sorts of *_EXPR ;) That's exactly the problem; it wouldn'

Re: RFA: Use gen_int_mode in plus_constant

2013-04-30 Thread Richard Biener
Richard Sandiford wrote: >This patch fixes out the GEN_INT/gen_int_mode that Richard pointed out >in the wide-int review. It also passes "mode" rather than "VOIDmode" >to immed_double_int_const. (As discussed in that thread, the latter >change shouldn't make any difference in practice, but is s

[gomp4] Some nesting region check adjustments

2013-04-30 Thread Jakub Jelinek
Hi! I've adjusted what I could now in check_omp_nesting_restrictions to match the 4.0 draft. 2013-04-30 Jakub Jelinek * omp-low.c (check_omp_nesting_restrictions): Diagnose OpenMP constructs nested inside simd region. Don't treat #pragma omp simd as work-sharing regio

[gomp4] Tiny c-family addition

2013-04-30 Thread Jakub Jelinek
Hi! Just one tiny omission (but neither of the parsers can parse the accelerator stuff yet). 2013-04-30 Jakub Jelinek * c-pragma.c (omp_pragmas): Add PRAGMA_OMP_DISTRIBUTE. --- gcc/c-family/c-pragma.c.jj 2013-03-27 13:01:09.0 +0100 +++ gcc/c-family/c-pragma.c 2013-04-30

[gomp4] Handle seq_cst clause on OpenMP atomics

2013-04-30 Thread Jakub Jelinek
Hi! Something I've missed in the latest draft, there can be seq_cst clause to request sequentially atomic operation as opposed to relaxed. Handled thusly: 2013-04-30 Jakub Jelinek * gimple-pretty-print.c (dump_gimple_omp_atomic_load, dump_gimple_omp_atomic_store): Handle gimp

Re: [PATCH, wwwdocs] Fix PR 50642

2013-04-30 Thread Joseph S. Myers
On Tue, 30 Apr 2013, Shakthi Kannan wrote: > Hi, > > - Original Message - > | From: "Joseph S. Myers" > | > | onlinedocs copies of manuals are generated by > | maintainer-scripts/update_web_docs_svn, which doesn't use any makefiles. > \-- > > Could you please list the pre-requisite ste

[Patch, ARM][11/n] Split patterns that output multiple assembly instructions - thumb2.md

2013-04-30 Thread Greta Yorsh
This patch continues to clean up patterns that output multiple assembly instructions. It handles most of the patterns in thumb2.md. The following patterns are not split: thumb2_movcond, thumb2_cond_move - complex, maybe later. tls_load_dot_plus_four - won't split: uses asm_out in output state

Re: [PATCH, fixincludes] Fix NULL on AIX

2013-04-30 Thread David Edelsohn
On Tue, Apr 30, 2013 at 11:28 AM, Bruce Korb wrote: > Hi David, > > Looks good to me, but for a small nit: > Unless there is some reason you want to emphasize tabs with the "\t" escapes. > I think the "here string" is easier to read. I coped c_fix_arg from openbsd_null_definition fix. I don't kn

Re: [C++ Patch/RFC] PR 57092

2013-04-30 Thread Paolo Carlini
... sorry about the latter confabulations ;) I went almost crazy trying to avoid the copy_node, but in this case it seems really necessary. The below appears to regtest fine (already beyond g++.dg/dg.exp). Thanks, Paolo. /// Index: cp/pt.c ==

RE: [PATCH][ARM] Fix PR 56809

2013-04-30 Thread Kyrylo Tkachov
> -Original Message- > From: Ramana Radhakrishnan > Sent: 03 April 2013 16:16 > To: Kyrylo Tkachov > Cc: GCC Patches > Subject: Re: [PATCH][ARM] Fix PR 56809 > > On 04/03/13 16:07, Kyrylo Tkachov wrote: > > Hi all, > > > > This patch fixes an ICE that we encounter when building gcc on arm

[Patch,ARM,Committed] Remove trailing whitespaces in thumb2.md

2013-04-30 Thread Greta Yorsh
Remove trailing whitespaces in thumb2.md. Committed as obvious (trunk r198464). Thanks, Greta 2013-04-30 Greta Yorsh * config/arm/thumb2.md: Remove trailing whitespaces.diff --git a/gcc/config/arm/thumb2.md b/gcc/config/arm/thumb2.md index 697350c..3aa7247 100644 --- a/gcc/config/arm/

[Patch, Fortran] Avoid segfault in conformable_arrays

2013-04-30 Thread Tobias Burnus
The issue was found by Reinhold Bader when testing the Fortran-Dev branch (thanks!), but it also affects GCC 4.6 and later. The patch is rather obvious: The segfault occured as tail->u.ar.start[i] == NULL. An alternative is to could use "continue" instead of "break". Build and regtested on x8

Re: [PATCH, fixincludes] Fix NULL on AIX

2013-04-30 Thread Bruce Korb
Hi, On Tue, Apr 30, 2013 at 9:42 AM, David Edelsohn wrote: > I coped c_fix_arg from openbsd_null_definition fix. I don't know if > two different formats for similar fixes would be confusing, but I do > not have a problem using the EOF version. I haven't reviewed all patches and I don't always ra

[PATCH,ARM] Internal memcpy using LDRD/STRD

2013-04-30 Thread Greta Yorsh
This patch for gcc's internal memcpy emits LDRD/STRD whenever possible, if prefer_ldrd_strd field is set in tune_params. It uses DImode moves in both ARM and Thumb modes. The generic move_by_pieces implementation cannot be used as is to generate the same instruction sequence. To handle cases in

Re: [AArch64] Improve description of CM instructions in RTL

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 15:55, James Greenhalgh wrote: > > Comparison operators can be described in RTL in a more accurate > manner than simply using UNSPECs. Do this. > > We need the split cases for DI mode to give reload a sensible > alternative. Without these, reload would churn out some t

Re: [AArch64] Remap neon vcmp functions to C/TREE

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 15:58, James Greenhalgh wrote: > > Where possible, describe these Neon intrinsics in C. Where not > possible, remap them to a builtin which will fold to an equivalent > C function. > > Add the support code to allow folding of these instructions. > > As GE_EXPR, GT_EXPR

Re: [AArch64] Add special case when expanding vcond with arms {-1, -1}, {0, 0}.

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 16:06, James Greenhalgh wrote: > > If the end goal of a VEC_COND_EXPR is to pick between > {-1, -1, -1, -1} and {0, 0, 0, 0} > then we do not need to do a bit select, this is just > a move of the generated mask to the result operand. > > This patch checks for this case

Re: [AArch64 Testsuite] Fix fallout from FCM changes.

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 16:08, James Greenhalgh wrote: > > Hi, > > gcc.target/aarch64/scalar_intrinsics.c uses the vcled_s64 style > intrinsics. As these now just map to a C operation, we need to first > ensure that the arguments to these functions make their way to the > SIMD register set. >

Re: [AArch64] Add combiner patterns for FAC instructions

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 16:11, James Greenhalgh wrote: > > Hi, > > This patch adds patterns to allow the combiner to generate > FAC instructions. > > Regression tested on aarch64-none-elf with no regressions. > > OK? > > Thanks, > James > > --- > gcc/ > > 2013-04-30 James Greenhalgh > >

Re: [AArch64] Rewrite vca Neon patterns in C.

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 16:13, James Greenhalgh wrote: > > Hi, > > This patch renames the vac neon intrinsics to vca > as required by the newest version of the neon intrinsics spec. > > While we are there, we also update all the vca* intrinsics to > a C implimentation. > > Regression tested o

Re: [AArch64] Add testcases for FAC, FCM changes.

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 16:15, James Greenhalgh wrote: > > Hi, > > This patch adds two testcases for the scalar and vector versions > of the vca* intrinsics. > > New tests pass on aarch64-none-elf. > > OK? > > Thanks, > James > > --- > gcc/testsuite/ > > 2013-04-30 James Greenhalgh > >

Re: [AArch64] Refactor vector max and min RTL and builtins.

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 16:19, James Greenhalgh wrote: > > Hi, > > This patch moves around the max/min infrastructure such that > we can more uniformly access the backend RTL patterns and such > that it uses the standard pattern names. > > We also fixup the bugs this introduces in arm_neon.h

Re: [AArch64] Fold max and min reduction builtins to tree.

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 16:21, James Greenhalgh wrote: > > Hi, > > This patch folds the signed versions of min and max reduction > builtins to tree form. We can't safely fold the unsigned > versions as they do not currently have the correct type. > > We must use TARGET_GIMPLE_FOLD_BUILTIN as

Re: [AArch64] Refactor reduc_plus patterns.

2013-04-30 Thread Marcus Shawcroft
OK /Marcus On 30 April 2013 16:42, James Greenhalgh wrote: > > Hi, > > This patch refactors the reduc_plus RTL patterns in the > AArch64 back-end. We do this to provide a more uniform > interface for arm_neon.h builtins. Because of this we can rewrite > the intrinsics in arm_neon.h to use these s

Re: [PATCH, fixincludes] Fix NULL on AIX

2013-04-30 Thread David Edelsohn
On Tue, Apr 30, 2013 at 1:09 PM, Bruce Korb wrote: > By analogy, I'd prefer the EOF version there, too. But ultimately, > it works and is not exactly illegible, so I have no serious objection > to the fix. It would be nice if it were combinable, but doing so is > tricky. I'm guessing that the

[patch] Update email, status in MAINTAINERS

2013-04-30 Thread Brooks Moses
I have a new email address (and accompanying new job), and I hope to be working with GCC in the new job at least a bit, so it's time to update my email address in the MAINTAINERS file. Also, it has been some years since I have touched the Fortran front end, so I have demoted myself from Fortran re

Re: [C++ Patch/RFC] PR 57092

2013-04-30 Thread Jason Merrill
On 04/30/2013 11:59 AM, Paolo Carlini wrote: Currently, in some cases (see, eg, template/canon-type-9.C) we have that id is true and DECLTYPE_TYPE_EXPR (t) is a TEMPLATE_PARM_INDEX but the tsubst_copy_and_build call returns a TEMPLATE_PARM_INDEX again, not an ADDR_EXPR, not an expression Good p

Re: [DWARF] Fix multiple register spanning location.

2013-04-30 Thread Cary Coutant
2013-04-26 Christian Bruel * dwarf2out.c (multiple_reg_loc_descriptor): Use DBX_REGISTER_NUMBER for spaning registers. s/spaning/spanning/ Index: dwarf2out.c === --- dwarf2out.c (revision 198287) +++ dwarf2out.c

Re: [C++ Patch/RFC] PR 57092

2013-04-30 Thread Paolo Carlini
Hi, On 04/30/2013 08:57 PM, Jason Merrill wrote: On 04/30/2013 11:59 AM, Paolo Carlini wrote: Currently, in some cases (see, eg, template/canon-type-9.C) we have that id is true and DECLTYPE_TYPE_EXPR (t) is a TEMPLATE_PARM_INDEX but the tsubst_copy_and_build call returns a TEMPLATE_PARM_INDEX

[google] Add function name to function_patch_* sections (issue9025045)

2013-04-30 Thread Harshit Chopra
Adding function name to the function_patch_* sections when -ffunction-sections is provided. Helps in garbage collecting dead functions with the help of linker. Tested: Tested using 'make -k check-gcc RUNTESTFLAGS="i386.exp=patch* --target_board=unix\{-m32,,-m64\}"'. 2013-04-30 Harshit Chopra

Re: [AArch64] Add special case when expanding vcond with arms {-1, -1}, {0, 0}.

2013-04-30 Thread Marc Glisse
On Tue, 30 Apr 2013, James Greenhalgh wrote: If the end goal of a VEC_COND_EXPR is to pick between {-1, -1, -1, -1} and {0, 0, 0, 0} then we do not need to do a bit select, this is just a move of the generated mask to the result operand. This patch checks for this case, and emits the appropriat

RE: [patch] cilkplus: Array notation for C patch

2013-04-30 Thread Bernhard Reutner-Fischer
On 30 April 2013 00:55:16 "Joseph S. Myers" wrote: > + if (flag_enable_cilkplus && contains_array_notation_expr (cond)) > +{ > + error_at (start_locus, "array notation expression cannot be used in a " > + "loop%'s condition"); > + return; > +} > + if (flag_enable_

[patch, fortran, committed] Change 1**k to 1

2013-04-30 Thread Thomas Koenig
Hello world, the attached patch was committed after Tobias Burnus' OK in PR 57073 and regression-testing. Thomas 2013-04-30 Thomas Koenig PR fortran/57071 * frontend-passes.c (optimize_power): Simplify 1**k to 1. 2013-04-30 Thomas Koenig PR fort

Re: [patch, fortran, committed] Change 1**k to 1

2013-04-30 Thread Steve Kargl
On Tue, Apr 30, 2013 at 11:47:46PM +0200, Thomas Koenig wrote: > + e->value.op.op1 = NULL; > + e->value.op.op2 = NULL; > + mpz_init_set_si (e->value.integer, 1); > + /* Typespec cand location are still OK. */ s/cand/and -- steve

[GOOGLE] Change function naming to use context function assembler name to replace function id

2013-04-30 Thread Dehao Chen
This patch changes to use context function name to replace function id, which is not available in AutoFDO builds. Bootstrapped and passed regression tests. OK for google branches? Thanks, Dehao Index: gcc/l-ipo.c === --- gcc/l-ipo.

[Google 4.8 Dwarf] Backport .debug_str in .o files with -gsplit-dwarf from trunk (issue9052046)

2013-04-30 Thread Sterling Augustine
The enclosed patch puts strings in the .debug_str section in the .o file with -gsplit-dwarf. It is backported from from trunk (which, in turn, was ported from google_4.7). http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01355.html The patch itself is no different from the one finally applied to trun

Re: [google] Add function name to function_patch_* sections (issue9025045)

2013-04-30 Thread Xinliang David Li
ok. David On Tue, Apr 30, 2013 at 1:34 PM, Harshit Chopra wrote: > Adding function name to the function_patch_* sections when > -ffunction-sections is provided. Helps in garbage collecting dead functions > with the help of linker. > > Tested: > Tested using 'make -k check-gcc RUNTESTFLAGS="i

Re: [Google 4.8 Dwarf] Backport .debug_str in .o files with -gsplit-dwarf from trunk (issue9052046)

2013-04-30 Thread Cary Coutant
> The enclosed patch puts strings in the .debug_str section in the .o > file with -gsplit-dwarf. It is backported from from trunk (which, in turn, > was ported from google_4.7). > > http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01355.html > > The patch itself is no different from the one finally app

Re: [Google 4.8 Dwarf] Backport .debug_str in .o files with -gsplit-dwarf from trunk (issue9052046)

2013-04-30 Thread Sterling Augustine
On Tue, Apr 30, 2013 at 4:47 PM, Cary Coutant wrote: >> The enclosed patch puts strings in the .debug_str section in the .o >> file with -gsplit-dwarf. It is backported from from trunk (which, in turn, >> was ported from google_4.7). >> >> http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01355.html >>

Re: [C++ Patch/RFC] PR 57092

2013-04-30 Thread Jason Merrill
On 04/30/2013 04:49 PM, Paolo Carlini wrote: Ok. Then shall we do something like the below? Yeah, just add a comment explaining that it's there to handle instantiated template non-type arguments. Jason

RFC: PATCH to avoid linking multiple front ends at once with parallel make

2013-04-30 Thread Jason Merrill
Since GNU Make doesn't support anything like the .MUTEX directive (http://savannah.gnu.org/bugs/?func=detailitem&item_id=17873), and accidentally doing "make -j8 -l4" makes my laptop useless for several minutes while it tries to link all the front ends at once, I decided to kludge a workaround.