Re: [PATCH] Add support for ARM embedded multilibs

2015-11-03 Thread Ramana Radhakrishnan
On Wed, Nov 4, 2015 at 12:29 AM, Jasmin J. wrote: > Thank you for your patch - In this case before you make any more changes to this patch - comparing your patch and Terry's patch here https://gcc.gnu.org/ml/gcc-patches/2014-05/msg00729.html shows no real differences, I would like to ask if you

Re: Add VIEW_CONVERT_EXPR to operand_equal_p

2015-11-03 Thread Jan Hubicka
> > This fails on ia64. > > gnat.dg/discr44.adb and gnat.dg/discr45.adb are supposed to fail everywhere > (and there are also a few ACATS failures everywhere). Sorry for the awkward > situation but they are testcases exposing the recent type system breakage. Are these supposed to be fixed by R

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Jonathan Wakely
On 4 November 2015 at 11:50, Jonathan Wakely wrote: > On 4 November 2015 at 02:11, Daniel Gutson wrote: >> Since this is a nothrow new, we thought that probably the system >> might not be exceptions-friendly (such as certain embedded systems), >> so we wanted to provide the new_handler the ability

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Jonathan Wakely
On 4 November 2015 at 02:11, Daniel Gutson wrote: > Since this is a nothrow new, we thought that probably the system > might not be exceptions-friendly (such as certain embedded systems), > so we wanted to provide the new_handler the ability to do something else > other than trying to allocate memo

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Jonathan Wakely
On 4 November 2015 at 01:55, Martin Sebor wrote: > On 11/03/2015 05:35 AM, Aurelio Remonda wrote: >> >> Currently, whenever operator new (std::nothrow) fails to allocate memory, >> it'll >> check if there is a new-handler function available. If there is, it'll >> call >> the handler and then try to

Division Optimization in match and simplify

2015-11-03 Thread Hurugalawadi, Naveen
Hi, Please find attached the patch that moves some division optimizations from fold-const using match and simplify. Please review the patch and let me know if any modifications are required. Regression tested the patch on X86 without any issues. Thanks, Naveen ChangeLog 2015-11-04 Naveen H.S

[openacc] acc loop updates in fortran

2015-11-03 Thread Cesar Philippidis
This patch updates the fortran front end so that it supports the acc loop clauses in a similar manner to both the c and c++ front ends in addition to addressing a couple of other loose ends. Specifically: * the tile clause now supports a list of arguments like c and c++ * made the default claus

Re: libgo patch committed: Update to Go 1.5 release

2015-11-03 Thread Ian Lance Taylor
On Tue, Nov 3, 2015 at 12:17 PM, Lynn A. Boger wrote: > I can see the problem in gotest. It has to do with the setting of GOARCH. > > Previously GOARCH was being set based on the --goarch argument, but now > goarch is being set. Not sure both GOARCH and goarch are needed? > > I've attached a pat

Re: [PATCH] S/390: Fix warning in "*movstr" pattern.

2015-11-03 Thread Dominik Vogt
On Tue, Nov 03, 2015 at 06:47:28PM +0100, Ulrich Weigand wrote: > Dominik Vogt wrote: > > > @@ -2936,7 +2936,7 @@ > > (set (mem:BLK (match_operand:P 1 "register_operand" "0")) > > (mem:BLK (match_operand:P 3 "register_operand" "2"))) > > (set (match_operand:P 0 "register_operand" "=d")

[PATCH] Add support for ARM embedded multilibs

2015-11-03 Thread Jasmin J.
Ported from svn://gcc.gnu.org/svn/gcc/branches/ARM/embedded-4_9-branch * config.gcc (--with-multilib-list): Accept arm embedded cores. * configure.ac (with_multilib_list): Export for being used in arm embedded multilib fragment. * configure: Regenerated. * Makefile.in (with_multilib_list):

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Martin Sebor
On 11/03/2015 04:08 PM, Mike Stump wrote: On Nov 3, 2015, at 1:10 PM, Martin Sebor wrote: The "as if" requirement implies that any observable effects of "the (possibly replaced) ordinary version" must be preserved. The repeated calls to the new handler are among such effects. Unless the stand

Re: [PATCH], PowerPC IEEE 128-bit patch #8 (libgcc bits)

2015-11-03 Thread Joseph Myers
On Tue, 3 Nov 2015, Michael Meissner wrote: > > Another thing that would help find missing libgcc functions is enabling > > libquadmath for powerpc (building with -mvsx) - though there isn't a > > libquadmath testsuite - missing functions would mean it fails to link > > (though this wouldn't fi

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Mike Stump
On Nov 3, 2015, at 1:10 PM, Martin Sebor wrote: > The "as if" requirement implies that any observable effects of > "the (possibly replaced) ordinary version" must be preserved. > The repeated calls to the new handler are among such effects. Unless the standard is fixed to say that one cannot obse

Re: [PATCH], PowerPC IEEE 128-bit patch #8 (libgcc bits)

2015-11-03 Thread Mike Stump
On Nov 3, 2015, at 2:17 PM, Michael Meissner wrote: > The intention was to keep those to allow other compilers (XL, LLVM) to use > these functions for developing their own IEEE 128-bit support. However, since > I posted the patch, those groups feel they cannot use the libgcc support due > to >

Re: [PATCH], PowerPC IEEE 128-bit patch #8 (libgcc bits)

2015-11-03 Thread Michael Meissner
I had the wrong email address for Tulio. -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797

Re: [PATCH], PowerPC IEEE 128-bit patch #8 (libgcc bits)

2015-11-03 Thread Michael Meissner
On Tue, Oct 27, 2015 at 11:38:48PM +, Joseph Myers wrote: > This patch does not take account of the issues with the powerpc-specific > functions that I pointed out in > when they > were posted to libc-alpha. (Testcases covering th

[openacc] tile, independent, default, private and firstprivate support in c/++

2015-11-03 Thread Cesar Philippidis
This patch does the following to the c and c++ front ends: * parsing support for the tile, independent, default (none), private and firstprivate clauses in c and c++ * updates c_oacc_split_loop_clauses to filter out the loop clauses from combined parallel/kernels loops The c front end al

Re: libgo patch committed: Update to Go 1.5 release

2015-11-03 Thread Andreas Schwab
Ian Lance Taylor writes: > I don't know that I have access to a big-endian PPC GNU/Linux machine any > more. There's gcc110 on the compile farm. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something c

Re: [ping] Fix PR debug/66728

2015-11-03 Thread Mike Stump
c -march=x86-64 -g -O -fstack-protector" ! .byte 0x1 # DW_AT_language .ascii "t.c\0" # DW_AT_name .long .LASF1 # DW_AT_comp_dir: "/home/mrs/net/gcc-linuxO0/gcc" .quad .Ltext0 # DW_AT_low_pc --- 28,35 .long .L

Re: [ARM] Fix PR middle-end/65958

2015-11-03 Thread Eric Botcazou
> Yes, but that's not usual, ARM and SPARC have it too, 4096 happens to be the > limit of reg+off addressing mode on several architectures. ... not unusual... -- Eric Botcazou

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Martin Sebor
Besides violating the requirement of the C++ standard, replacing Could you please point us to the relevant section where this behavior is enforced? We couldn't find it so far. The Required behavior of the nothrow operator new reads: This nothrow version of operator new returns a pointer obt

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Daniel Gutson
On Tue, Nov 3, 2015 at 5:25 PM, Martin Sebor wrote: > On 11/03/2015 05:35 AM, Aurelio Remonda wrote: >> >> Currently, whenever operator new (std::nothrow) fails to allocate memory, >> it'll >> check if there is a new-handler function available. If there is, it'll >> call >> the handler and then tr

[PATCH], Add power9 support to GCC, patch #1

2015-11-03 Thread Michael Meissner
This patch adds the stub support to allow users to use -mcpu=power9, so that in the future it will generate code for the Power9 systems (ISA 3.0). At this time, the stub only sets up the switches. Future patches to GCC 6.x and later GCC 7.x will add support for various features in power9. I have

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Martin Sebor
On 11/03/2015 05:35 AM, Aurelio Remonda wrote: Currently, whenever operator new (std::nothrow) fails to allocate memory, it'll check if there is a new-handler function available. If there is, it'll call the handler and then try to allocate again. Otherwise, it'll return a null pointer. This ret

Re: libgo patch committed: Update to Go 1.5 release

2015-11-03 Thread Lynn A. Boger
I can see the problem in gotest. It has to do with the setting of GOARCH. Previously GOARCH was being set based on the --goarch argument, but now goarch is being set. Not sure both GOARCH and goarch are needed? I've attached a patch I used to get it to work. It worked for ppc64le and ppc64

[PATCH] [graphite] improve debug of codegen

2015-11-03 Thread Sebastian Pop
From: Aditya Kumar - fix printing of ISL stmt and parameter names - move dot_scop* functions outside of anonymous namespace. * graphite-isl-ast-to-gimple.c: Include tree-cfg.h. (translate_isl_ast_node_user): Add more dumps: call print_loops_bb. * graphite-scop-detection.c (dot_all_scops_1)

Re: [PATCH] c/67882 - improve -Warray-bounds for invalid offsetof

2015-11-03 Thread Martin Sebor
I guess this is a case where I could say either "I wrote the patch" or "I requested changes to a patch in review"; in the latter case I can approve it. Joseph seems on board with what we've discussed, so I'd say please wait until Tuesday for objections then commit. I didn't get to committing the

Re: [PATCH] Fix declaration of pthread-structs in s-osinte-rtems.ads

2015-11-03 Thread Arnaud Charlet
> > Your ChangeLog entry is not in the proper format, see sections 6.8.1 and > > 6.8.2 from http://www.gnu.org/prep/standards/standards.html > > > > The diff itself is OK. > > Ok, fixed this. See the new diff below. This is now OK, you can go ahead and commit it. > > You can use svn merge to me

[PATCH, libcpp]: Correctly advance the pointer to an aligned address.

2015-11-03 Thread Uros Bizjak
Hello! The formulae to advance the pointer to an aligned address is: /* This macro rounds x up to the y boundary. */ #define ROUND_UP(x,y) (((x) + (y) - 1) & ~((y) - 1)) Another example of missing "-1" can be found in libcpp, lex.c/search_line_sse42. Luckily, we already checked for unaligned in

[PATCH] PR67518 and PR53852 -- add testcase.

2015-11-03 Thread VandeVondele Joost
Attached testcases for two previously fixed PRs (and thanks to Dominique who was quicker for PR67982). 2015-11-03 Joost VandeVondele PR middle-end/53852 PR middle-end/67518 * gfortran.dg/PR67518.f90: New test. * gfortran.dg/PR53852.f90: New test. OK for trunk afte

OpenACC dimension range propagation optimization

2015-11-03 Thread Nathan Sidwell
Richard, this patch implements VRP for the 2 openacc axis internal fns I've added. We know the position within a dimension cannot exceed that dimensions extend. Further, if the extend is dynamic, the target backend may well know there's a hardware-mandated maximum value. Hence, added a new

[PATCH,committed] PR fortran/67982 -- add testcase

2015-11-03 Thread Dominique d'Humières
I've committed a testcase for PR fortran/67982 to trunk. Dominique Index: gcc/testsuite/gfortran.dg/warn_unused_function_3.f90 === --- gcc/testsuite/gfortran.dg/warn_unused_function_3.f90(nonexistent) +++ gcc/testsuite/gfortran

Re: [ARM] Fix PR middle-end/65958

2015-11-03 Thread Eric Botcazou
> Unless there really is common code between the two patches, this should > be separated out into two posts, one for ARM and one for AArch64. The ARM bits were approved by Ramana and installed right away. > Hmm, so if PROBE_INTERVAL != 4096 we barf! Yes, but that's not usual, ARM and SPARC have

Re: [AArch64] Update comments on the usage of X30 in FIXED_REGISTERS and CALL_USED_REGISTERS

2015-11-03 Thread Jiong Wang
On 02/11/15 14:52, Richard Earnshaw wrote: On 02/11/15 12:58, Jiong Wang wrote: On 02/11/15 12:01, Richard Earnshaw wrote: On 16/10/15 15:36, Jiong Wang wrote: The patch https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02654.html from last year changed the definition of LR in CALL_USED_REGISTE

Re: [PATCH] S/390: Fix warning in "*movstr" pattern.

2015-11-03 Thread Ulrich Weigand
Dominik Vogt wrote: > @@ -2936,7 +2936,7 @@ > (set (mem:BLK (match_operand:P 1 "register_operand" "0")) > (mem:BLK (match_operand:P 3 "register_operand" "2"))) > (set (match_operand:P 0 "register_operand" "=d") > - (unspec [(mem:BLK (match_dup 1)) > + (unspec:P [(mem:BLK (mat

Re: [PATCH] Fix declaration of pthread-structs in s-osinte-rtems.ads

2015-11-03 Thread Jan Sommer
Am Tuesday 03 November 2015, 18:10:53 schrieb Arnaud Charlet: > > Let's try again. This time I made the diff against trunk with the changes > > Sebastian recommended, included a ChangeLog and used svn-diff. > > If this patch goes through, please let me know how the backporting works. > > Your Chan

Re: [ARM] Fix PR middle-end/65958

2015-11-03 Thread Richard Earnshaw
On 06/10/15 11:11, Eric Botcazou wrote: >> Thanks - I have no further comments on this patch. We probably need to >> implement the same on AArch64 too in order to avoid similar problems. > > Here's the implementation for aarch64, very similar but simpler since there > is > no shortage of scratch

Re: libgo patch committed: Update to Go 1.5 release

2015-11-03 Thread Ian Lance Taylor
On Tue, Nov 3, 2015 at 7:48 AM, Lynn A. Boger wrote: > > We are seeing failures on all the libgo tests when gccgo is built with the > latest trunk > on ppc64 (BE) and when running the testsuite for 64 bit. The failures > do not occur if run on ppc64 BE with m32 and do not occur on ppc64le. > > Th

[PATCH] S/390: Fix warning in "*movstr" pattern.

2015-11-03 Thread Dominik Vogt
The attached patch fixes the message "warning: dest missing a mode?" from s390.md. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.md ("*movstr"): Fix warning. >From e03251cacf2004e4cb302de03b44bb1a3f6ad827 Mon Sep 17 00:00:00 2001 From: Dominik Vog

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Aurelio Remonda
On Tue, Nov 3, 2015 at 10:26 AM, Paolo Carlini wrote: > Finally, since you are touching acinclude.m4 you should > normally run autoreconf, mention in the ChangeLog the changed regenerated > files and eventually commit those changes too (like the ChangeLog entries, > those aren't normally part of t

Re: [PATCH][RFC] Remove warning for SET VOIDmode -> BLKmode.

2015-11-03 Thread Dominik Vogt
(Debug code removed from patch.) Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * genrecog.c (validate_pattern): Allow "set VOIDmode -> BLKmode" without warnings. >From 04376919c108c42a2e9835dd1809b198bc47513f Mon Sep 17 00:00:00 2001 From: Dominik Vogt Date:

Re: [PATCH] Fix declaration of pthread-structs in s-osinte-rtems.ads

2015-11-03 Thread Arnaud Charlet
> Let's try again. This time I made the diff against trunk with the changes > Sebastian recommended, included a ChangeLog and used svn-diff. > If this patch goes through, please let me know how the backporting works. Your ChangeLog entry is not in the proper format, see sections 6.8.1 and 6.8.2 fr

Re: [Patch AArch64] Switch constant pools to separate rodata sections.

2015-11-03 Thread James Greenhalgh
On Tue, Nov 03, 2015 at 04:36:45PM +, Ramana Radhakrishnan wrote: > Hi, > > Now that PR63304 is fixed and we have an option to address > any part of the memory using adrp / add or adrp / ldr instructions > it makes sense to switch out literal pools into their own > mergeable sections by

[PATCH][RFC] Remove warning for SET VOIDmode -> BLKmode.

2015-11-03 Thread Dominik Vogt
The attached patch removes the messages "warning: source missing a mode?" and "warning: operand ... missing mode?" (genrecog.c) for the case that the DEST of a SET rtx has BKLmode and SRC has void mode. The mvcle instruction on s390 has a pretty weird format that takes the lowest eight bits of an

[c++-delayed-folding] Introduce convert_to_real_nofold

2015-11-03 Thread Marek Polacek
The last piece for convert.c. Since convert_to_real uses fold () rather than fold_buildN, I defined a new macro to keep the code more compact. With this committed, convert.c should be dealt with. If there's anything else I could help with, please let me know. Bootstrapped/regtested on x86_64-li

Re: [PATCH] remove unused config/arm/coff.h

2015-11-03 Thread Richard Earnshaw
On 03/11/15 15:37, Trevor Saunders wrote: > On Tue, Nov 03, 2015 at 06:49:20AM -0700, Jeff Law wrote: >> On 11/03/2015 04:31 AM, tbsaunde+...@tbsaunde.org wrote: >>> From: Trevor Saunders >>> >>> Hi, >>> >>> $subject, nothing refers to this header so we might as well remove it. >>> >>> tested I ca

Re: [PATCH, 2/2] Handle recursive restrict in function parameter

2015-11-03 Thread Tom de Vries
On 03/11/15 16:08, Richard Biener wrote: On Tue, 3 Nov 2015, Tom de Vries wrote: On 01/11/15 19:20, Tom de Vries wrote: On 01/11/15 19:03, Tom de Vries wrote: So, the new patch series is: 1Rename make_restrict_var_constraints to make_param_constraints 2Handle recursive

[Patch AArch64] Switch constant pools to separate rodata sections.

2015-11-03 Thread Ramana Radhakrishnan
Hi, Now that PR63304 is fixed and we have an option to address any part of the memory using adrp / add or adrp / ldr instructions it makes sense to switch out literal pools into their own mergeable sections by default. This would mean that by default we could now start getting the benefit

Re: [OpenACC] declare directive

2015-11-03 Thread James Norris
On 10/27/2015 03:18 PM, James Norris wrote: Hi! This patch adds the processing of OpenACC declare directive in C and C++. (Note: Support in Fortran is already in trunk.) Commentary on the changes is included as an attachment (NOTES). All of the code is in the gomp-4_0-branch

Re: [PATCH] PR 68192 Export AIX TLS symbols

2015-11-03 Thread David Edelsohn
On Tue, Nov 3, 2015 at 9:47 AM, Richard Biener wrote: > On Tue, Nov 3, 2015 at 3:23 PM, David Edelsohn wrote: >> TLS symbols in AIX display a new, different symbol type in nm output. >> Libtool explicitly creates a list of exported symbols for shared >> libraries using nm and does not recognize t

Re: [2/2] OpenACC routine support

2015-11-03 Thread Jakub Jelinek
On Tue, Nov 03, 2015 at 10:56:37AM -0500, Nathan Sidwell wrote: > On 11/03/15 10:38, Jakub Jelinek wrote: > >On Mon, Nov 02, 2015 at 02:23:19PM -0500, Nathan Sidwell wrote: > >>Here are the tests for the routine support. The compiler tests check > >>invalid combinations of gang, worker, vector & s

Re: [2/2] OpenACC routine support

2015-11-03 Thread Nathan Sidwell
On 11/03/15 10:38, Jakub Jelinek wrote: On Mon, Nov 02, 2015 at 02:23:19PM -0500, Nathan Sidwell wrote: Here are the tests for the routine support. The compiler tests check invalid combinations of gang, worker, vector & seq. The libgomp execution tests check the expected partioning occurs with

Re: [PATCH] Fix typo.

2015-11-03 Thread Yulia Koval
Here it is. gcc/ * config/i386/i386.c (m_SKYLAKE_AVX512): Fix typo. Yulia On Tue, Nov 3, 2015 at 5:32 PM, Uros Bizjak wrote: > On Tue, Nov 3, 2015 at 3:16 PM, Yulia Koval wrote: >> Hi, >> >> This patch fixes a typo: PROCESSOT -> PROCESSOR. Ok for trunk? > > Trivial patch, OK with a suitabl

Re: [Boolean Vector, patch 1/5] Introduce boolean vector to be used as a vector comparison type

2015-11-03 Thread Jeff Law
On 10/29/2015 07:08 AM, Ilya Enkovich wrote: On 28 Oct 22:37, Ilya Enkovich wrote: Seems the problem occurs in this check in expand_vector_operations_1: /* A scalar operation pretending to be a vector one. */ if (VECTOR_BOOLEAN_TYPE_P (type) && !VECTOR_MODE_P (TYPE_MODE (type))

Re: [1/3] OpenACC reductions

2015-11-03 Thread Nathan Sidwell
On 11/03/15 10:46, Jakub Jelinek wrote: On Mon, Nov 02, 2015 at 11:18:37AM -0500, Nathan Sidwell wrote: This is the core execution bits of OpenACC reductions. We have a new internal fn 'IFN_GOACC_REDUCTION' and a new target hook goacc.reduction, to lower it on the target compiler. So, let me

Re: [1/2] OpenACC routine support

2015-11-03 Thread Nathan Sidwell
On 11/03/15 10:35, Jakub Jelinek wrote: On Mon, Nov 02, 2015 at 02:21:43PM -0500, Nathan Sidwell wrote: --- gcc/c/c-parser.c(revision 229667) +++ gcc/c/c-parser.c(working copy) @@ -1160,7 +1160,8 @@ enum c_parser_prec { static void c_parser_external_declaration (c_parser *); static v

[PATCH] Refactor BB vectorization

2015-11-03 Thread Richard Biener
This refactors BB vectorization in preparation to make it work on sub-BB granularity. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2015-11-03 Richard Biener * tree-vect-data-refs.c (vect_analyze_data_refs): Do not collect data references he

Re: [PATCH] replace BITS_PER_UNIT with __CHAR_BIT__ in target libs

2015-11-03 Thread Trevor Saunders
On Tue, Nov 03, 2015 at 01:20:26PM +, Joseph Myers wrote: > On Mon, 2 Nov 2015, Jeff Law wrote: > > > Based on Bernd's comments, I think this is fine. Any sense of how much work > > there is left to cleanup the runtime's inclusion of gcc's config/ target > > headers? > > See

Re: libgo patch committed: Update to Go 1.5 release

2015-11-03 Thread Lynn A. Boger
We are seeing failures on all the libgo tests when gccgo is built with the latest trunk on ppc64 (BE) and when running the testsuite for 64 bit. The failures do not occur if run on ppc64 BE with m32 and do not occur on ppc64le. The messages say this: make[3]: Entering directory `/home/boger/g

Re: [1/3] OpenACC reductions

2015-11-03 Thread Jakub Jelinek
On Mon, Nov 02, 2015 at 11:18:37AM -0500, Nathan Sidwell wrote: > This is the core execution bits of OpenACC reductions. > > We have a new internal fn 'IFN_GOACC_REDUCTION' and a new target hook > goacc.reduction, to lower it on the target compiler. So, let me start with a few questions: 1) does

[PATCH][AArch64][v2] Improve comparison with complex immediates followed by branch/cset

2015-11-03 Thread Kyrill Tkachov
Hi all, This patch slightly improves sequences where we want to compare against a complex immediate and branch against the result or perform a cset on it. This means transforming sequences of mov+movk+cmp+branch into sub+subs+branch. Similar for cset. Unfortunately I can't just do this by simply

Re: Multiply Optimization in match and Simplify

2015-11-03 Thread Richard Biener
On Tue, Nov 3, 2015 at 6:12 AM, Hurugalawadi, Naveen wrote: > Hi, > > Thanks for the review and suggestions. > >>> Please do not drop A - B -> A + (-B) from fold-const as match.pd >>> doesn't implement all of fold-const.c negate_expr_p support. > > Done. > >>> which is more expensive. This means

Re: [2/2] OpenACC routine support

2015-11-03 Thread Jakub Jelinek
On Mon, Nov 02, 2015 at 02:23:19PM -0500, Nathan Sidwell wrote: > Here are the tests for the routine support. The compiler tests check > invalid combinations of gang, worker, vector & seq. The libgomp execution > tests check the expected partioning occurs within loops. As with the > reduction t

Re: [PATCH] remove unused config/arm/coff.h

2015-11-03 Thread Trevor Saunders
On Tue, Nov 03, 2015 at 06:49:20AM -0700, Jeff Law wrote: > On 11/03/2015 04:31 AM, tbsaunde+...@tbsaunde.org wrote: > >From: Trevor Saunders > > > >Hi, > > > >$subject, nothing refers to this header so we might as well remove it. > > > >tested I can still build on x86_64-linux-gnu, not that I wou

Re: [1/2] OpenACC routine support

2015-11-03 Thread Jakub Jelinek
On Mon, Nov 02, 2015 at 02:21:43PM -0500, Nathan Sidwell wrote: > --- gcc/c/c-parser.c (revision 229667) > +++ gcc/c/c-parser.c (working copy) > @@ -1160,7 +1160,8 @@ enum c_parser_prec { > static void c_parser_external_declaration (c_parser *); > static void c_parser_asm_definition (c_parser *

Re: libgo patch committed: Update to Go 1.5 release

2015-11-03 Thread Ian Lance Taylor
On Mon, Nov 2, 2015 at 11:48 PM, Uros Bizjak wrote: > >> I have committed a patch to libgo to update it to the Go 1.5 release. >> >> As usual for libgo updates, the actual patch is too large to attach to >> this e-mail message. I've attached the changes to the gccgo-specific >> files. >> >> Boots

Re: [PATCH, 2/2] Handle recursive restrict in function parameter

2015-11-03 Thread Richard Biener
On Tue, 3 Nov 2015, Tom de Vries wrote: > On 01/11/15 19:20, Tom de Vries wrote: > > On 01/11/15 19:03, Tom de Vries wrote: > > > So, the new patch series is: > > > > > > 1Rename make_restrict_var_constraints to make_param_constraints > > > 2Handle recursive restrict in functi

Re: [v3 PATCH] Make the default constructors of tuple and pair conditionally explicit.

2015-11-03 Thread Ville Voutilainen
On 3 November 2015 at 16:42, Jonathan Wakely wrote: > On 3 November 2015 at 02:37, Paolo Carlini wrote: >> Hi, >> >> On 11/02/2015 09:20 PM, Ville Voutilainen wrote: >>> >>> On 2 November 2015 at 21:20, Paolo Carlini >>> wrote: Can we follow the terse style already used elsewhere (eg, >

Re: [PATCH] PR 68192 Export AIX TLS symbols

2015-11-03 Thread Richard Biener
On Tue, Nov 3, 2015 at 3:23 PM, David Edelsohn wrote: > TLS symbols in AIX display a new, different symbol type in nm output. > Libtool explicitly creates a list of exported symbols for shared > libraries using nm and does not recognize the new TLS symbols, so > those symbols are not exported. > >

Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-03 Thread Richard Biener
On Tue, Nov 3, 2015 at 3:13 PM, Martin Liška wrote: > On 11/03/2015 02:46 PM, Richard Biener wrote: >> On Fri, Oct 30, 2015 at 1:53 PM, Martin Liška wrote: >>> On 10/30/2015 01:13 PM, Richard Biener wrote: So I suggest to do the push/pop of cfun there. do_per_function_toporder can be ma

Re: [v3 PATCH] Make the default constructors of tuple and pair conditionally explicit.

2015-11-03 Thread Jonathan Wakely
On 3 November 2015 at 02:37, Paolo Carlini wrote: > Hi, > > On 11/02/2015 09:20 PM, Ville Voutilainen wrote: >> >> On 2 November 2015 at 21:20, Paolo Carlini >> wrote: >>> >>> Can we follow the terse style already used elsewhere (eg, >>> __is_direct_constructible_new_safe) thus directly inherit fr

Re: [gomp4, committed] Implement -foffload-alias

2015-11-03 Thread Tom de Vries
On 03/11/15 15:19, Tom de Vries wrote: I've dropped the two testcases from this patch, I'll commit in a follow-up patch. Committed to gomp-4_0-branch, as attached. Thanks, - Tom Add goacc/kernels-loop-offload-alias-{none,ptr}.c 2015-11-03 Tom de Vries * c-c++-common/goacc/kernels-loop-of

Re: [0/7] Type promotion pass and elimination of zext/sext

2015-11-03 Thread Richard Biener
On Mon, Nov 2, 2015 at 10:17 AM, Kugan wrote: > > > On 29/10/15 02:45, Richard Biener wrote: >> On Tue, Oct 27, 2015 at 1:50 AM, kugan >> wrote: >>> >>> >>> On 23/10/15 01:23, Richard Biener wrote: On Thu, Oct 22, 2015 at 12:50 PM, Kugan wrote: > > > > On 21/10/15

Re: [PATCH] Fix typo.

2015-11-03 Thread Uros Bizjak
On Tue, Nov 3, 2015 at 3:16 PM, Yulia Koval wrote: > Hi, > > This patch fixes a typo: PROCESSOT -> PROCESSOR. Ok for trunk? Trivial patch, OK with a suitable ChangeLog. Uros.

[PATCH/RFTesting][MIPS] Migrate reduction optabs in mips-ps-3d.md

2015-11-03 Thread Alan Lawrence
There are still a few uses of the old reduc_[us](plus|min|max)_ optabs remaining. This migrates the instances in mips-ps-3d.md. This seemed straightforward, as mips-ps-3d.md also provides a vec_extractv2sf. I tried to be conservative and handle all the possible cases for endianness, this may be ov

[PATCH][i386]Migrate reduction optabs to reduc__scal

2015-11-03 Thread Alan Lawrence
This migrates the various reduction optabs in sse.md to use the reduce-to-scalar form. I took the straightforward approach (equivalent to the migration code in expr.c/optabs.c) of generating a vector temporary, using the existing code to reduce to that, and extracting lane 0, in each pattern. Boot

Re: [gomp4 06/14] omp-low: copy omp_data_o to shared memory on NVPTX

2015-11-03 Thread Alexander Monakov
Hello, Here's an alternative patch that does not depend on exposure of shared-memory address space, and does not try to use pass_late_lower_omp. It's based on Bernd's suggestion to transform (use .omp_data_o) GOMP_parallel (fn, &omp_data_o, ...); .omp_data_o = {CLOBBER}; to .omp_data_o

[PATCH] PR 68192 Export AIX TLS symbols

2015-11-03 Thread David Edelsohn
TLS symbols in AIX display a new, different symbol type in nm output. Libtool explicitly creates a list of exported symbols for shared libraries using nm and does not recognize the new TLS symbols, so those symbols are not exported. This is a regression for TLS support on AIX. This patch updates

[PATCH] Fix declaration of pthread-structs in s-osinte-rtems.ads

2015-11-03 Thread Jan Sommer
Hi, Let's try again. This time I made the diff against trunk with the changes Sebastian recommended, included a ChangeLog and used svn-diff. If this patch goes through, please let me know how the backporting works. Best regards, Jan Index: ChangeLog =

[gomp4, committed] Implement -foffload-alias

2015-11-03 Thread Tom de Vries
[ was: Re: [gomp4, WIP] Implement -foffload-alias ] On 28/09/15 17:38, Tom de Vries wrote: Hi, this work-in-progress patch implements a new option -foffload-alias=. The option -foffload-alias=none instructs the compiler to assume that objects references and pointer dereferences in an offload

Re: [patch 4/3] Header file reduction - Tools for contrib - second cut

2015-11-03 Thread Andrew MacLeod
On 11/03/2015 09:00 AM, Jeff Law wrote: yeah, the reducer still needs some tweaks to be generally runnable I think. IN particular, how to deal with externally supplied macros it cant really see. Im still thinking about that one. Well, the solution is obvious, we continue the move away from

Re: [PATCH] remove unused config/arm/coff.h

2015-11-03 Thread Richard Earnshaw
On 03/11/15 13:49, Jeff Law wrote: > On 11/03/2015 04:31 AM, tbsaunde+...@tbsaunde.org wrote: >> From: Trevor Saunders >> >> Hi, >> >> $subject, nothing refers to this header so we might as well remove it. >> >> tested I can still build on x86_64-linux-gnu, not that I would expect >> anything >> e

[PATCH] Fix typo.

2015-11-03 Thread Yulia Koval
Hi, This patch fixes a typo: PROCESSOT -> PROCESSOR. Ok for trunk? Yulia patch Description: Binary data

Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-03 Thread Martin Liška
On 11/03/2015 02:46 PM, Richard Biener wrote: > On Fri, Oct 30, 2015 at 1:53 PM, Martin Liška wrote: >> On 10/30/2015 01:13 PM, Richard Biener wrote: >>> So I suggest to do the push/pop of cfun there. >>> do_per_function_toporder can be made static btw. >>> >>> Richard. >> >> Right, I've done that

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Andreas Schwab
Aurelio Remonda writes: > aurelio-remonda@Remonda-PC:~/gcc/libstdc++-v3$ autoreconf > configure.ac:74: error: Autoconf version 2.65 or higher is required Make sure you have automake 1.11.6. Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1

RE: [PATCH, MIPS, PR/61114] Migrate to reduc_..._scal optabs.

2015-11-03 Thread Moore, Catherine
> -Original Message- > From: Simon Dardis [mailto:simon.dar...@imgtec.com] > Sent: Wednesday, October 07, 2015 6:51 AM > To: Alan Lawrence; Matthew Fortune; Moore, Catherine > Cc: gcc-patches@gcc.gnu.org > Subject: RE: [PATCH, MIPS, PR/61114] Migrate to reduc_..._scal optabs. > > On the

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Aurelio Remonda
On Tue, Nov 3, 2015 at 10:26 AM, Paolo Carlini wrote: > Hi, > > On 11/03/2015 01:35 PM, Aurelio Remonda wrote: >> >> diff --git a/ChangeLog b/ChangeLog >> index 5b16ca2..a1cd0d3 100644 >> --- a/ChangeLog >> +++ b/ChangeLog >> @@ -1,3 +1,12 @@ >> +2015-10-30 Aurelio Remonda >> + >> + * lib

Re: [PATCH 6/6] Make SRA replace constant-pool loads

2015-11-03 Thread Richard Biener
On Thu, Oct 29, 2015 at 8:18 PM, Alan Lawrence wrote: > This has changed quite a bit since the previous revision > (https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01484.html), mostly due to Ada > and specifically Ada on ARM. > > I didn't find a good alternative to scanning for constant-pool accesse

Re: [patch 4/3] Header file reduction - Tools for contrib - second cut

2015-11-03 Thread Jeff Law
On 11/03/2015 06:24 AM, Andrew MacLeod wrote: Do you have any sense of whether or not coverage of the tools has improved over short time since we started squashing out conditional compilation? I was running the header file reordering bits on the trunk and was a bit surprised of how many things

[gomp4,committed] Handle recursive restrict in function parameter

2015-11-03 Thread Tom de Vries
[ was: Re: [PATCH, 2/2] Handle recursive restrict in function parameter ] On 03/11/15 14:46, Tom de Vries wrote: On 01/11/15 19:20, Tom de Vries wrote: On 01/11/15 19:03, Tom de Vries wrote: So, the new patch series is: 1Rename make_restrict_var_constraints to make_param_constraints

Re: [PATCH] remove unused config/arm/coff.h

2015-11-03 Thread Jeff Law
On 11/03/2015 04:31 AM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders Hi, $subject, nothing refers to this header so we might as well remove it. tested I can still build on x86_64-linux-gnu, not that I would expect anything else or that it is particularly relevent, ok? Trev gcc/Cha

Re: [PATCH, 2/2] Handle recursive restrict in function parameter

2015-11-03 Thread Tom de Vries
On 01/11/15 19:20, Tom de Vries wrote: On 01/11/15 19:03, Tom de Vries wrote: So, the new patch series is: 1Rename make_restrict_var_constraints to make_param_constraints 2Handle recursive restrict in function parameter I'll repost in reply to this message. This patch ad

Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-03 Thread Richard Biener
On Fri, Oct 30, 2015 at 1:53 PM, Martin Liška wrote: > On 10/30/2015 01:13 PM, Richard Biener wrote: >> So I suggest to do the push/pop of cfun there. >> do_per_function_toporder can be made static btw. >> >> Richard. > > Right, I've done that and it works (bootstrap has been currently running), >

Re: [PATCH] Use signed boolean type for boolean vectors

2015-11-03 Thread Richard Biener
On Wed, Oct 28, 2015 at 4:30 PM, Ilya Enkovich wrote: > 2015-10-28 18:21 GMT+03:00 Richard Biener : >> On Wed, Oct 28, 2015 at 2:13 PM, Ilya Enkovich >> wrote: >>> Hi, >>> >>> Testing boolean vector conversions I found several runtime regressions >>> and investigation showed it's due to incorrec

Re: [Boolean Vector, patch 1/5] Introduce boolean vector to be used as a vector comparison type

2015-11-03 Thread Jeff Law
On 11/03/2015 04:26 AM, Richard Biener wrote: On Mon, Nov 2, 2015 at 8:41 PM, Jeff Law wrote: On 10/29/2015 07:08 AM, Ilya Enkovich wrote: On 28 Oct 22:37, Ilya Enkovich wrote: Seems the problem occurs in this check in expand_vector_operations_1: /* A scalar operation pretending to be

Re: [PATCH] remove unused config/arm/coff.h

2015-11-03 Thread Richard Earnshaw
On 03/11/15 11:31, tbsaunde+...@tbsaunde.org wrote: > From: Trevor Saunders > > Hi, > > $subject, nothing refers to this header so we might as well remove it. > > tested I can still build on x86_64-linux-gnu, not that I would expect anything > else or that it is particularly relevent, ok? > O

Re: [mask-load, patch 1/2] Use boolean predicate for masked loads and store

2015-11-03 Thread Richard Biener
On Wed, Oct 28, 2015 at 4:23 PM, Ilya Enkovich wrote: > On 23 Oct 13:36, Ilya Enkovich wrote: >> 2015-10-23 13:32 GMT+03:00 Richard Biener : >> > >> > No, we'd get >> > >> > mask_1 = bool != 1; >> > >> > and the 'mask' variable should have been simplified to 'bool' >> > (yes, we'd insert a dead

Re: [PATCH] PR/67682, break SLP groups up if only some elements match

2015-11-03 Thread Richard Biener
On Tue, Oct 27, 2015 at 6:38 PM, Alan Lawrence wrote: > On 26/10/15 15:04, Richard Biener wrote: >> >> >> apart from the fact that you'll post a new version you need to adjust >> GROUP_GAP. >> You also seem to somewhat "confuse" "first I stmts" and "a group of >> size I", those >> are not the same

Re: [PATCH] Add configure flag for operator new (std::nothrow)

2015-11-03 Thread Paolo Carlini
Hi, On 11/03/2015 01:35 PM, Aurelio Remonda wrote: diff --git a/ChangeLog b/ChangeLog index 5b16ca2..a1cd0d3 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2015-10-30 Aurelio Remonda + + * libstdc++-v3/acinclude.m4: add enable_new_opnt_no_allocation_retry + flag definiti

Re: [patch 4/3] Header file reduction - Tools for contrib - second cut

2015-11-03 Thread Andrew MacLeod
On 11/03/2015 01:06 AM, Jeff Law wrote: On 10/14/2015 09:14 AM, Andrew MacLeod wrote: Here's the latest version of the tools for a sub directory in contrib. I've handled all the feedback, except I have not fully commented the python code in the tools, nor followed any particular coding conventio

  1   2   >