[PATCH] Fix PR68248

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-11-09 Richard Biener PR tree-optimization/68248 * tree-vect-generic.c (expand_vector_operations_1): Handle scalar rhs2. * gcc.dg/torture/pr68248.c: New testcase.

Re: [Patch] Change to argument promotion in fixed conversion library calls

On Fri, Nov 6, 2015 at 8:14 PM, Bernd Schmidt wrote: > On 11/06/2015 08:04 PM, Steve Ellcey wrote: >> >> When I made this change I had one regression in the GCC testsuite >> (gcc.dg/fixed-point/convert-sat.c). I tracked this down to the >> fact that

[PATCH] Improve BB vectorization dependence analysis

Currently BB vectorization computes all dependences inside a BB region and fails all vectorization if it cannot handle some of them. This is obviously not needed - BB vectorization can restrict the dependence tests to those that are needed to apply the load/store motion effectively performed by

[PATCH] 02/N Fix memory leaks in IPA

Hi. Following changes were consulted with Martin Jambor to properly release memory in IPA. It fixes leaks which popped up in tramp3d with -O2. Bootstrap and regression tests have been running. Ready after it finishes? Thanks, Martin >From 85b63f738030dd7a901c228ba76e24f820d31c5d Mon Sep 17

Re: [AArch64][PATCH 6/7] Add NEON intrinsics vqrdmlah and vqrdmlsh.

On 09/11/15 13:31, Christophe Lyon wrote: On 30 October 2015 at 16:52, Matthew Wahab wrote: On 30/10/15 12:51, Christophe Lyon wrote: On 23 October 2015 at 14:26, Matthew Wahab wrote: The ARMv8.1 architecture extension adds two

Re: [vec-cmp, patch 3/6] Vectorize comparison

On Mon, Nov 9, 2015 at 1:07 PM, Ilya Enkovich wrote: > On 26 Oct 16:09, Richard Biener wrote: >> On Wed, Oct 14, 2015 at 6:12 PM, Ilya Enkovich >> wrote: >> > + >> > + ops.release (); >> > + vec_defs.release (); >> >> No

Re: [vec-cmp, patch 2/6] Vectorization factor computation

On Mon, Nov 9, 2015 at 2:54 PM, Ilya Enkovich wrote: > 2015-10-20 16:45 GMT+03:00 Richard Biener : >> On Wed, Oct 14, 2015 at 1:21 PM, Ilya Enkovich >> wrote: >>> 2015-10-13 16:37 GMT+03:00 Richard Biener

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

On 06/11/15 12:55, Richard Biener wrote: > >> + /* GROUP_GAP of the first group now has to skip over the second group >> too. */ >> + GROUP_GAP (first_vinfo) += group2_size; > > Please add a MSG_NOTE debug printf stating that we split the group and > at which element. Done. > I think you

[patch] backport PIE support for FreeBSD to gcc-49

Hi, any objections that I apply this patch to gcc-4.9? It is FreeBSD only. TIA, Andreas 2015-11-09 Andreas Tobler Backport from mainline 2015-05-18 Andreas Tobler * config/freebsd-spec.h (FBSD_STARTFILE_SPEC): Add

Re: [PATCH 1/2] s/390: Implement "target" attribute.

On 11/02/2015 09:44 AM, Dominik Vogt wrote: > (@Uli: I'd like to hear your opinion on this issue. > Original message: > https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03403.html). > > On Fri, Oct 30, 2015 at 03:09:39PM +0100, Andreas Krebbel wrote: >> Why do we need x_s390_arch_specified and

Re: [vec-cmp, patch 2/6] Vectorization factor computation

2015-10-20 16:45 GMT+03:00 Richard Biener : > On Wed, Oct 14, 2015 at 1:21 PM, Ilya Enkovich wrote: >> 2015-10-13 16:37 GMT+03:00 Richard Biener : >>> On Thu, Oct 8, 2015 at 4:59 PM, Ilya Enkovich

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

On 03 Nov 14:42, Richard Biener wrote: > 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, >

Re: OpenACC Firstprivate

On Mon, Nov 09, 2015 at 08:59:15AM -0500, Nathan Sidwell wrote: > >This I'm afraid performs often two copies rather than just one (one to copy > >the host value to the present_copyin mapped value, another one in the > >region), > > I don't think that can be avoided. The host doesn't have control

Re: [PATCH] 02/N Fix memory leaks in IPA

On Mon, Nov 9, 2015 at 2:29 PM, Martin Liška wrote: > Hi. > > Following changes were consulted with Martin Jambor to properly release > memory in IPA. It fixes leaks which popped up in tramp3d with -O2. > > Bootstrap and regression tests have been running. > > Ready after it

Re: Extend tree-call-cdce to calls whose result is used

Hi, On Sat, 7 Nov 2015, Richard Sandiford wrote: > For -fmath-errno, builtins.c currently expands calls to sqrt to: > > y = sqrt_optab (x); > if (y != y) > [ sqrt (x); or errno = EDOM; ] > > - the call to sqrt is protected by the result of the optab rather > than the input. It

Re: [PATCH v2 11/13] Test case for conversion from __seg_tls:0

On Tue, Oct 20, 2015 at 11:27 PM, Richard Henderson wrote: > --- > gcc/testsuite/gcc.target/i386/addr-space-3.c | 10 ++ > 1 file changed, 10 insertions(+) > create mode 100644 gcc/testsuite/gcc.target/i386/addr-space-3.c > > diff --git

Re: [PATCH 4b/4] [ARM] PR63870 Remove error for invalid lane numbers

On 08/11/15 00:26, charles.bay...@linaro.org wrote: > From: Charles Baylis > > Charles Baylis > > * config/arm/neon.md (neon_vld1_lane): Remove error for invalid > lane number. > (neon_vst1_lane): Likewise. >

Re: [vec-cmp, patch 4/6] Support vector mask invariants

On Mon, Nov 9, 2015 at 1:11 PM, Ilya Enkovich wrote: > On 26 Oct 16:21, Richard Biener wrote: >> On Wed, Oct 14, 2015 at 6:13 PM, Ilya Enkovich >> wrote: >> > - val = fold_unary (VIEW_CONVERT_EXPR, TREE_TYPE (type), val); >> > +

Re: Add a combined_fn enum

Bernd Schmidt writes: > On 11/09/2015 11:24 AM, Richard Sandiford wrote: >> Bernd Schmidt writes: >>> I see it's already acked, but have you considered just doing away with >>> the builtin/internal function distinction? >> >> I think they're too

Re: [PATCH] Fix memory leaks and use a pool_allocator

On Mon, Nov 09, 2015 at 01:11:48PM +0100, Richard Biener wrote: > On Mon, Nov 9, 2015 at 12:22 PM, Martin Liška wrote: > > Hi. > > > > This is follow-up of changes that Richi started on Friday. > > > > Patch can bootstrap on x86_64-linux-pc and regression tests are running. > > >

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

On 11/04/2015 02:39 AM, Dominik Vogt wrote: > 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"))) >>>

Re: [PATCH] Fix memory leaks and use a pool_allocator

On 11/09/2015 01:11 PM, Richard Biener wrote: > On Mon, Nov 9, 2015 at 12:22 PM, Martin Liška wrote: >> Hi. >> >> This is follow-up of changes that Richi started on Friday. >> >> Patch can bootstrap on x86_64-linux-pc and regression tests are running. >> >> Ready for trunk? > >

Re: OpenACC Firstprivate

On Sat, Nov 07, 2015 at 08:50:28AM -0500, Nathan Sidwell wrote: > Index: gcc/gimplify.c > === > --- gcc/gimplify.c(revision 229892) > +++ gcc/gimplify.c(working copy) > @@ -108,9 +108,15 @@ enum omp_region_type >/* Data

Re: RFC: C++ delayed folding merge

On 11/09/2015 04:24 AM, Eric Botcazou wrote: One question: The branch changes 'convert' to not fold its result, and it's not clear to me whether that's part of the expected behavior of a front end 'convert' function or not. I don't think that you should change the behavior for front-ends that

Re: OpenACC Firstprivate

On 11/09/15 08:46, Jakub Jelinek wrote: On Sat, Nov 07, 2015 at 08:50:28AM -0500, Nathan Sidwell wrote: Index: gcc/gimplify.c === If you want to switch to hexadecimal, you should change all values in the enum to hexadecimal for

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

On Mon, Nov 9, 2015 at 3:03 PM, Ilya Enkovich wrote: > On 03 Nov 14:42, Richard Biener wrote: >> On Wed, Oct 28, 2015 at 4:30 PM, Ilya Enkovich >> wrote: >> > 2015-10-28 18:21 GMT+03:00 Richard Biener : >> >> On Wed,

RE: [RFC][PATCH] Preferred rename register in regrename pass

Hi Bernd, Sorry for late reply. The updated patch was bootstrapped on x86_64-unknown-linux-gnu and cross tested on mips-img-linux-gnu using r229786. The results below were generated for CSiBE benchmark and the numbers in columns express bytes in format 'net (gain/loss)' to show the difference

Re: [AArch64][PATCH 6/7] Add NEON intrinsics vqrdmlah and vqrdmlsh.

On 30 October 2015 at 16:52, Matthew Wahab wrote: > On 30/10/15 12:51, Christophe Lyon wrote: >> >> On 23 October 2015 at 14:26, Matthew Wahab >> wrote: >>> >>> The ARMv8.1 architecture extension adds two Adv.SIMD instructions, >>> sqrdmlah

Re: [PATCH] Minor refactoring in tree-ssanames.c & freelists verifier

Hi, On Mon, 9 Nov 2015, Jeff Law wrote: +verify_ssaname_freelists (struct function *fun) +{ + /* Do nothing if we are in RTL format. */ + basic_block bb; + FOR_EACH_BB_FN (bb, fun) +{ + if (bb->flags & BB_RTL) + return; +} gimple_in_ssa_p (fun); + /* Then note the

GCC 6 Status Report (2015-11-09)

Status == We've pushed back the switch to Stage 3 to the end of Saturday, Nov 14th. This is to allow smooth draining of review queues. Quality Data Priority # Change from last report --- --- P14+ 2 P2

[PATCH][optabs][ifcvt][1/3] Define negcc, notcc optabs

Hi all, This is a rebase of the patch I posted at: https://gcc.gnu.org/ml/gcc-patches/2015-09/msg00154.html The patch has been ok'd by Jeff but I wanted to hold off committing it until my fixes for the ifcvt regressions on sparc and x86_64 were fixed. The rebase conflicts were due to Richard's

[patch] backport remove of soft float for FreeBSD powerpc for gcc-4.9

Hi all, any objections when I apply the below patch to gcc-4.9? TIA, Andreas 2015-11-09 Andreas Tobler Backport from mainline 2015-03-04 Andreas Tobler * config/rs6000/t-freebsd64: Remove 32-bit soft-float multilibs.

Re: OpenACC Firstprivate

On 11/09/15 08:59, Nathan Sidwell wrote: On 11/09/15 08:46, Jakub Jelinek wrote: On Sat, Nov 07, 2015 at 08:50:28AM -0500, Nathan Sidwell wrote: Say int arr[64]; // initialize arr #pragma acc data copyin (arr) { // modify arr on the host # pragma acc parallel firstprivate (arr)

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

On 08/11/15 11:42, Andreas Schwab wrote: > This is causing a bootstrap comparison failure in gcc/go/gogo.o. I'm looking into this - this is now PR68256. regards Ramana > > Andreas. >

Re: [PATCH] Fix memory leaks and use a pool_allocator

On Mon, Nov 9, 2015 at 2:26 PM, Martin Liška wrote: > On 11/09/2015 01:11 PM, Richard Biener wrote: >> On Mon, Nov 9, 2015 at 12:22 PM, Martin Liška wrote: >>> Hi. >>> >>> This is follow-up of changes that Richi started on Friday. >>> >>> Patch can bootstrap on

Re: OpenACC Firstprivate

On 11/09/15 09:10, Jakub Jelinek wrote: On Mon, Nov 09, 2015 at 08:59:15AM -0500, Nathan Sidwell wrote: This I'm afraid performs often two copies rather than just one (one to copy the host value to the present_copyin mapped value, another one in the region), I don't think that can be avoided.

Re: Add null identifiers to genmatch

On Mon, Nov 9, 2015 at 12:17 AM, Jeff Law wrote: > On 11/07/2015 07:31 AM, Pedro Alves wrote: >> >> Hi Richard, >> >> Passerby comment below. >> >> On 11/07/2015 01:21 PM, Richard Sandiford wrote: >>> >>> -/* Lookup the identifier ID. */ >>> +/* Lookup the identifier ID. Allow

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On Nov 9, 2015, at 11:46 AM, Jeff Law wrote: On 11/09/2015 12:38 PM, Bernd Schmidt wrote: >> We might want to think about making a policy decision to try waiving >> some of the testing requirements for target macro -> hook conversions. >> Maybe try only a "build to cc1"

Re: [Patch] Change to argument promotion in fixed conversion library calls

On Mon, 2015-11-09 at 21:47 +0100, Bernd Schmidt wrote: > On 11/09/2015 05:59 PM, Steve Ellcey wrote: > > Here is a version with the code moved into a new function. How does > > this look? > > > > 2015-11-09 Steve Ellcey > > > > * optabs.c (prepare_libcall_arg): New

Re: [PATCH], Add power9 support to GCC, patch #5 (ISA 3.0 fusion)

On Mon, Nov 9, 2015 at 11:57 AM, Segher Boessenkool wrote: > On Mon, Nov 09, 2015 at 12:34:20PM -0500, Michael Meissner wrote: >> > > +(define_insn "*toc_fusionload_" >> > > + [(set (match_operand:QHSI 0 "int_reg_operand" "=,??r") >> > > + (match_operand:QHSI 1

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On Mon, Nov 09, 2015 at 12:46:33PM -0700, Jeff Law wrote: > On 11/09/2015 12:38 PM, Bernd Schmidt wrote: > >On 11/09/2015 07:52 PM, Trevor Saunders wrote: > > > >>yeah, that's more or less my thought, and this makes hookization easier > >>since you can now mechanically add a hook for each thing in

Re: [PATCH 05/12] always define VMS_DEBUGGING_INFO

On Mon, Nov 09, 2015 at 08:37:19PM +0100, Bernd Schmidt wrote: > On 11/09/2015 08:29 PM, Trevor Saunders wrote: > >as I said in 0/12 this did go through config-list.mk, and checking again > >this does build on alpha-dec-vms. > > The question I have is - why does it build on any other target? It's

Re: [PATCH], Add power9 support to GCC, patch #5 (ISA 3.0 fusion)

On Mon, Nov 09, 2015 at 01:11:41PM -0800, David Edelsohn wrote: > On Mon, Nov 9, 2015 at 11:57 AM, Segher Boessenkool > wrote: > > On Mon, Nov 09, 2015 at 12:34:20PM -0500, Michael Meissner wrote: > >> > > +(define_insn "*toc_fusionload_" > >> > > + [(set

Re: [PATCH], Add power9 support to GCC, patch #7 (direct move enhancements)

I evidently forgot to attach the patch. [gcc] 2015-11-08 Michael Meissner * config/rs6000/constraints.md (we constraint): New constraint for 64-bit power9 vector support. (wL constraint): New constraint for the element in a vector that

Re: Extend tree-call-cdce to calls whose result is used

Hi, On Mon, 9 Nov 2015, Richard Sandiford wrote: > +static bool > +can_use_internal_fn (gcall *call) > +{ > + /* Only replace calls that set errno. */ > + if (!gimple_vdef (call)) > +return false; Oh, I managed to confuse this in my head while reading the patch. So, hmm, you don't

Remove instantiations when no concept check

Hi I just committed this trivial cleanup. 2015-11-09 François Dumont * include/bits/stl_algo.h (partial_sort_copy): Instantiate std::iterator_traits only if concept checks. (lower_bound): Likewise. (upper_bound): Likewise. (equal_range):

Remove unused openacc call

I've committed this to trunk. It nuke the now unused GOACC_GET_NUM_THREADS and GOACC_GET_THREAD_NUM calls. Also fixed up some comment typos I noticed nathan 2015-11-09 Nathan Sidwell * omp-low.c: Fix some OpenACC comment typos. (lower_reduction_clauses):

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On 11/09/2015 02:30 PM, Trevor Saunders wrote: So in general when I've done cross target things I think I've found more bugs with config-list.mk than with a regtest, but the regtest has found some things I think. I'm finding config-list.mk fairly reliable, with the notable exception of the

Re: RFC: C++ delayed folding merge

> Right, the change is just to the C++ front end 'convert'. OK, thanks for the clarification. -- Eric Botcazou

Re: [1/2] OpenACC routine support

On 11/09/2015 04:31 PM, Nathan Sidwell wrote: > 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 { >>>

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On Mon, 9 Nov 2015, Trevor Saunders wrote: > The add default macro definitions then wrap those with hooks, then > target by target replace the macro by hook overrides approach seems to > provide that you can incrementally test and fiind most of the issues, > but the change a macro every where

Re: [PATCH], Add power9 support to GCC, patch #4

On Mon, Nov 09, 2015 at 10:29:10AM -0600, Segher Boessenkool wrote: > On Sun, Nov 08, 2015 at 07:39:14PM -0500, Michael Meissner wrote: > > +;; Pretend we have a memory form of extswsli until register allocation is > > done > > +;; so that we use LWZ to load the value from memory, instead of LWA.

Re: [PATCH], Add power9 support to GCC, patch #5 (ISA 3.0 fusion)

On Mon, Nov 09, 2015 at 11:16:27AM -0600, Segher Boessenkool wrote: > On Sun, Nov 08, 2015 at 07:42:04PM -0500, Michael Meissner wrote: > > - /* Power8 currently will only do the fusion if the top 11 bits of the > > addis > > - value are all 1's or 0's. */ > >value = INTVAL (int_const);

[PATCH, 6/16] Add pass_oacc_kernels

On 09/11/15 16:35, Tom de Vries wrote: Hi, this patch series for stage1 trunk adds support to: - parallelize oacc kernels regions using parloops, and - map the loops onto the oacc gang dimension. The patch series contains these patches: 1Insert new exit block only when needed in

[gomp4] remove IFN_GOACC_DIM handling from device_lower

I've committed this to gomp4, the relevant handling is in gimple-fold now. nathan 2015-11-09 Nathan Sidwell * omp-low.c (oacc_xform_dim): Delete. (execute_oacc_device_lower): Remove IFN_GOACC_DIM_POS, IFN_GOACC_DIM_SIZE handling. Index: omp-low.c

Re: [PATCH v3 2/2] [PR debug/67192] Further fix C loops' back-jump location

On Sat, Nov 07 2015, Jeff Law wrote: > Also OK. And please consider using those tests with the C++ compiler > to see if it's suffering from the same problem. Not really, but there's still an issue. In the C front-end the back-jump's location of an unconditional loop was sometimes set to the

Re: [PATCH], Add power9 support to GCC, patch #3 (scalar count trailing zeros)

On Sun, Nov 8, 2015 at 4:37 PM, Michael Meissner wrote: > This patch adds support for scalar count trailing zeros instruction that is > being added to ISA 3.0 (power9). > > I have built this patch (along with patches #2 and #4) with a bootstrap build > on a power8

Re: Extend tree-call-cdce to calls whose result is used

Michael Matz writes: > On Mon, 9 Nov 2015, Richard Sandiford wrote: > >> -ffast-math would already cause us to treat the function as not setting >> errno, so the code wouldn't be used. > > What is "the code"? I don't see any checking of the relevant flags in > tree-call-cdce.c,

[PATCH, 7/16] Add pass_dominator_oacc_kernels

On 09/11/15 16:35, Tom de Vries wrote: Hi, this patch series for stage1 trunk adds support to: - parallelize oacc kernels regions using parloops, and - map the loops onto the oacc gang dimension. The patch series contains these patches: 1Insert new exit block only when needed in

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On 11/09/2015 05:47 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * defaults.h (EH_RETURN_HANDLER_RTX): New default definition. * df-scan.c

Re: [PATCH 05/12] always define VMS_DEBUGGING_INFO

In general I think the _DEBUGGING_INFO patches are going to be OK, modulo Jeff's comment about stage 1. I think they shouldn't have been split - it causes numerous unnecessary extra changes, and the intermediate stages look very inconsistent. -#ifdef VMS_DEBUGGING_INFO - else if

[PATCH, 8/16] Add pass_ch_oacc_kernels

On 09/11/15 16:35, Tom de Vries wrote: Hi, this patch series for stage1 trunk adds support to: - parallelize oacc kernels regions using parloops, and - map the loops onto the oacc gang dimension. The patch series contains these patches: 1Insert new exit block only when needed in

Re: [ping] Fix PR debug/66728

On Nov 6, 2015, at 5:06 AM, Richard Biener wrote: >> If there are no substantial reasons to not check it in now, I’d like to >> proceed and get it checked in. People can refine it further in tree if they >> want. Any objections? > > Ok with a changelog entry and

Re: [PATCH 01/12] reduce conditional compilation for HARD_FRAME_POINTER_IS_ARG_POINTER

On 11/09/2015 05:47 PM, tbsaunde+...@tbsaunde.org wrote: +++ b/gcc/dbxout.c @@ -3076,10 +3076,8 @@ dbxout_symbol_location (tree decl, tree type, const char *suffix, rtx home) || (REG_P (XEXP (home, 0)) && REGNO (XEXP (home, 0)) != HARD_FRAME_POINTER_REGNUM

Re: [PATCH][AArch64] PR target/68129: Define TARGET_SUPPORTS_WIDE_INT

On 9 November 2015 at 11:32, Kyrill Tkachov wrote: > 2015-11-09 Kyrylo Tkachov > > PR target/68129 > * config/aarch64/aarch64.h (TARGET_SUPPORTS_WIDE_INT): Define to 1. > * config/aarch64/aarch64.c (aarch64_print_operand,

Re: [PATCH][AArch64] PR target/68129: Define TARGET_SUPPORTS_WIDE_INT

On 9 November 2015 at 15:45, Kyrill Tkachov wrote: > > On 09/11/15 15:34, Marcus Shawcroft wrote: >> >> On 9 November 2015 at 11:32, Kyrill Tkachov >> wrote: >> >>> 2015-11-09 Kyrylo Tkachov >>> >>> PR target/68129

[PATCH 2/6] Make builtin_vectorized_function take a combined_fn

This patch replaces the fndecl argument to builtin_vectorized_function with a combined_fn and gets the vectoriser to call it for internal functions too. The patch also moves vectorisation of machine-specific built-ins to a new hook, builtin_md_vectorized_function. I've attached a -b version too

[PATCH 03/12] remove conditional compilation of sdb debug info

From: Trevor Saunders We need to include gsyms.h before tm.h because some targets (rl78 iirc) define macros that conflict with identifiers in gsyms.h. This means sdbout.c won't produce correct output for those targets, but it previously couldn't either because it

[PATCH 08/12] always define DWARF2_LINENO_DEBUGGING_INFO

From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * defaults.h (DWARF2_LINENO_DEBUGGING_INFO): new default definition. * dwarf2out.c (dwarf2out_init): Adjust. * opts.c (set_debug_level):

[PATCH 05/12] always define VMS_DEBUGGING_INFO

From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * defaults.h (VMS_DEBUGGING_INFO): New default definition. * doc/tm.texi: Regenerate. * doc/tm.texi.in: Adjust. * dwarf2out.c

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On Mon, 2015-11-09 at 11:47 -0500, tbsaunde+...@tbsaunde.org wrote: > From: Trevor Saunders > > gcc/ChangeLog: > > 2015-11-09 Trevor Saunders > > * defaults.h (EH_RETURN_HANDLER_RTX): New default definition. > * df-scan.c

Re: [PATCH], Add power9 support to GCC, patch #2 (add modulus instructions)

On Mon, Nov 09, 2015 at 09:48:50AM -0600, Segher Boessenkool wrote: > Hi, > > On Sun, Nov 08, 2015 at 07:36:16PM -0500, Michael Meissner wrote: > > [gcc/testsuite] > > * lib/target-supports.exp (check_p9vector_hw_available): Add > > checks for power9 availability. > >

Re: [PATCH 05/12] always define VMS_DEBUGGING_INFO

On 11/09/2015 11:34 AM, Bernd Schmidt wrote: In general I think the _DEBUGGING_INFO patches are going to be OK, modulo Jeff's comment about stage 1. I think they shouldn't have been split - it causes numerous unnecessary extra changes, and the intermediate stages look very inconsistent.

Re: [PATCH 02/12] remove EXTENDED_SDB_BASIC_TYPES

The last target using this was i960, which was removed many years ago, so there's no reason to keep it. gcc/ChangeLog: 2015-11-09 Trevor Saunders * gsyms.h (enum sdb_type): Remove code for EXTENDED_SDB_BASIC_TYPES. (enum sdb_masks):

Re: [PATCH], Add power9 support to GCC, patch #3 (scalar count trailing zeros)

On Mon, Nov 09, 2015 at 09:59:43AM -0600, Segher Boessenkool wrote: > On Sun, Nov 08, 2015 at 07:37:53PM -0500, Michael Meissner wrote: > > This patch adds support for scalar count trailing zeros instruction that is > > being added to ISA 3.0 (power9). > > I bet you should change

Re: [PATCH 12/12] always define ENABLE_OFFLOADING

On 11/09/2015 05:47 PM, tbsaunde+...@tbsaunde.org wrote: -#ifdef ENABLE_OFFLOADING /* If the user didn't specify any, default to all configured offload targets. */ if (offload_targets == NULL) handle_foffload_option (OFFLOAD_TARGETS); -#endif This one I would keep guarded

Re: [PATCH 00/12] misc conditional compilation work

On Mon, Nov 09, 2015 at 10:57:10AM -0700, Jeff Law wrote: > On 11/09/2015 09:47 AM, tbsaunde+...@tbsaunde.org wrote: > >From: Trevor Saunders > > > >Hi, > > > >basically $subject, making some code unconditionally compiled, and changing > >other things from #ifdef to #if

Re: [PATCH v2 11/13] Test case for conversion from __seg_tls:0

Hi! On Mon, 9 Nov 2015 15:46:20 +0100, Richard Biener wrote: > On Tue, Oct 20, 2015 at 11:27 PM, Richard Henderson wrote: > > --- /dev/null > > +++ b/gcc/testsuite/gcc.target/i386/addr-space-3.c > > @@ -0,0 +1,10 @@ > > +/* { dg-do compile } */ > >

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On 11/09/2015 07:42 PM, Jeff Law wrote: On 11/09/2015 11:27 AM, Bernd Schmidt wrote: On 11/09/2015 05:47 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * defaults.h

Re: [PATCH], Add power9 support to GCC, patch #5 (ISA 3.0 fusion)

On Sun, Nov 8, 2015 at 4:42 PM, Michael Meissner wrote: > This patch adds support for new fusion forms in ISA 3.0 (power9). In > particular, ISA 3.0 can fuse GPR loads of R0, FPR loads, GPR stores, FPR > stores, and some constant generation that ISA 2.07 (power8)

Re: RFC: Incomplete Draft Patches to Correct Errors in Loop Unrolling Frequencies (bugzilla problem 68212)

On 11/07/2015 03:44 PM, Kelvin Nilsen wrote: This is a draft patch to partially address the concerns described in bugzilla problem report https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68212). The patch is incomplete in the sense that there are some known shortcomings with nested loops which I am

Re: [PATCH 5/6] Simplify rs6000_builtin_vectorized_function

On Mon, Nov 9, 2015 at 8:30 AM, Richard Sandiford wrote: > After the previous patches it's no longer necessary for > TARGET_BUILTIN_VECTORIZED_FUNCTION to return functions that > map to the vector optab of the original operation. We'll use > a vector form of the

Re: [PATCH 11/12] always define HAVE_AS_LEB128

-#ifdef HAVE_AS_LEB128 +#if HAVE_AS_LEB128 This patch doesn't seem to actually remove any conditional compilation? Bernd

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On 11/09/2015 11:27 AM, Bernd Schmidt wrote: On 11/09/2015 05:47 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * defaults.h (EH_RETURN_HANDLER_RTX): New default definition.

Re: [PATCH][AArch64] PR target/68129: Define TARGET_SUPPORTS_WIDE_INT

On Nov 9, 2015, at 3:32 AM, Kyrill Tkachov wrote: > The aarch64 port does not define TARGET_SUPPORTS_WIDE_INT. > Ok for trunk and GCC 5? :-) I’d endorse it, but, best left to the target folks.

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

On Mon, Nov 09, 2015 at 11:42:19AM -0700, Jeff Law wrote: > On 11/09/2015 11:27 AM, Bernd Schmidt wrote: > >On 11/09/2015 05:47 PM, tbsaunde+...@tbsaunde.org wrote: > >>From: Trevor Saunders > >> > >>gcc/ChangeLog: > >> > >>2015-11-09 Trevor Saunders

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

On Mon, Nov 09, 2015 at 04:46:01PM +, Ramana Radhakrishnan wrote: > > > On 08/11/15 11:42, Andreas Schwab wrote: > > This is causing a bootstrap comparison failure in gcc/go/gogo.o. > > > > Andreas. > > > > I've had a look at this for sometime this afternoon and the trigger is the >

Re: [PATCH 00/12] misc conditional compilation work

On 11/09/2015 09:47 AM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders Hi, basically $subject, making some code unconditionally compiled, and changing other things from #ifdef to #if so they can be made unconditional incrementally. patches individually

Re: [PATCH], Add power9 support to GCC, patch #4

On Sun, Nov 8, 2015 at 4:39 PM, Michael Meissner wrote: > This patch adds support for the EXTSWSLI instruction that is being added to > PowerPC ISA 3.0 (power9). > > I have built this patch (along with patches #2 and #3) with a bootstrap build > on a power8 little

Re: [PATCH][AArch64] PR target/68129: Define TARGET_SUPPORTS_WIDE_INT

On 09/11/15 15:34, Marcus Shawcroft wrote: On 9 November 2015 at 11:32, Kyrill Tkachov wrote: 2015-11-09 Kyrylo Tkachov PR target/68129 * config/aarch64/aarch64.h (TARGET_SUPPORTS_WIDE_INT): Define to 1. *

[PATCH, 2/16] Make create_parallel_loop return void

On 09/11/15 16:35, Tom de Vries wrote: Hi, this patch series for stage1 trunk adds support to: - parallelize oacc kernels regions using parloops, and - map the loops onto the oacc gang dimension. The patch series contains these patches: 1Insert new exit block only when needed in

Re: [PATCH], Add power9 support to GCC, patch #1 (revised)

On Sun, Nov 8, 2015 at 4:33 PM, Michael Meissner wrote: > This is patch #1 that I revised. I changed -mfusion-toc to -mtoc-fusion. I > changed the references to ISA 2.08 to 3.0. I added two new debug switches for > code in future patches that in undergoing

[PATCH, 5/16] Add in_oacc_kernels_region in struct loop

On 09/11/15 16:35, Tom de Vries wrote: Hi, this patch series for stage1 trunk adds support to: - parallelize oacc kernels regions using parloops, and - map the loops onto the oacc gang dimension. The patch series contains these patches: 1Insert new exit block only when needed in

[PATCH 6/6] Simplify aarch64_builtin_vectorized_function

After the previous patches it's no longer necessary for TARGET_BUILTIN_VECTORIZED_FUNCTION to return functions that map to the vector optab of the original operation. We'll use a vector form of the internal function instead. gcc/ * config/aarch64/aarch64-builtins.c

Re: [OpenACC] declare directive

Jakub, On 11/09/2015 10:21 AM, Jakub Jelinek wrote: On Mon, Nov 09, 2015 at 10:01:32AM -0600, James Norris wrote: + if (lookup_attribute ("omp declare target", DECL_ATTRIBUTES (decl))) Here you only look up "omp declare target", not "omp declare target link". So, what happens if you mix

[PATCH 11/12] always define HAVE_AS_LEB128

From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * acinclude.m4: Always define HAVE_AS_LEB128. * configure: Regenerate. * configure.ac: Adjust. * dwarf2asm.c (dw2_asm_output_data_uleb128):

[PATCH 3/6] Vectorize internal functions

This patch tries to vectorize built-in and internal functions as internal functions first, falling back on the current built-in target hooks otherwise. gcc/ * internal-fn.h (direct_internal_fn_info): Add vectorizable flag. * internal-fn.c (direct_internal_fn_array): Update

[PATCH 07/12] always define DBX_DEBUGGING_INFO

From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * config/arc/arc.h: Define DBX_DEBUGGING_INFO to 1. * config/pdp11/pdp11.h: Likewise. * defaults.h (DBX_DEBUGGING_INFO): New default definition.

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

On 08/11/15 11:42, Andreas Schwab wrote: > This is causing a bootstrap comparison failure in gcc/go/gogo.o. > > Andreas. > I've had a look at this for sometime this afternoon and the trigger is the aarch64_use_constant_blocks_p change which appears to be causing a bootstrap comparison

[PATCH series, 16] Use parloops to parallelize oacc kernels regions

Hi, this patch series for stage1 trunk adds support to: - parallelize oacc kernels regions using parloops, and - map the loops onto the oacc gang dimension. The patch series contains these patches: 1 Insert new exit block only when needed in transform_to_exit_first_loop_alt

  1   2   3   >