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

2015-11-09 Thread David Edelsohn
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 endian system. There were no reg

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

2015-11-09 Thread Richard Sandiford
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, so I wonder what

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

2015-11-09 Thread David Edelsohn
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 little endian system. There were

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

2015-11-09 Thread Andreas Arnez
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 tok

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

2015-11-09 Thread Jeff Law
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 bootstrapped + regtested on x86_

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

2015-11-09 Thread Thomas Schwinge
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 } */ > > +/* { dg-options "-O" } */ > > +/* { dg-final

Re: [PATCH 5/6] Simplify rs6000_builtin_vectorized_function

2015-11-09 Thread David Edelsohn
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 internal function instead. > > > g

Re: [PATCH 10/12] always define EH_RETURN_HANDLER_RTX

2015-11-09 Thread David Malcolm
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 (df_get_exit_block_use_set): Adjust. > * except.c (e

[gomp4] remove IFN_GOACC_DIM handling from device_lower

2015-11-09 Thread Nathan Sidwell
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 ==

[PATCH, 6/16] Add pass_oacc_kernels

2015-11-09 Thread Tom de Vries
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: RFC: Incomplete Draft Patches to Correct Errors in Loop Unrolling Frequencies (bugzilla problem 68212)

2015-11-09 Thread Bernd Schmidt
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 s

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

2015-11-09 Thread Michael Meissner
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);

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

2015-11-09 Thread Michael Meissner
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 AArch64] Switch constant pools to separate rodata sections.

2015-11-09 Thread James Greenhalgh
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 > aarch6

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

2015-11-09 Thread Michael Meissner
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 CTZ_DEFINED_VALUE

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

2015-11-09 Thread Segher Boessenkool
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); >if ((value & (HOST_WIDE_INT)0x) != 0) Space after cast, like (HOST_WID

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

2015-11-09 Thread Michael Matz
Hi, 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, so I wonder what would prevent the addition

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

2015-11-09 Thread Robert Suchanek
Hi, > On 11/09/2015 02:32 PM, Robert Suchanek wrote: > > The results below were generated for CSiBE benchmark and the numbers in > > columns express bytes in format 'net (gain/loss)' to show the difference > > with and without the patch when -frename-registers switch is used. > > I'm not entirel

Re: [hsa 5/12] New HSA-related GCC options

2015-11-09 Thread Martin Jambor
Hi, On Fri, Nov 06, 2015 at 09:42:25AM +0100, Richard Biener wrote: > On Thu, 5 Nov 2015, Martin Jambor wrote: > > > Hi, > > > > the following small part of the merge deals with new options. It adds > > four independent things: > > > > 1) flag_disable_hsa is used by code in opts.c (in the firs

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

2015-11-09 Thread Steve Ellcey
On Fri, 2015-11-06 at 20:29 +0100, Bernd Schmidt wrote: > On 11/06/2015 08:27 PM, Steve Ellcey wrote: > > > > Are you thinking of a simple function that is called on all targets or a > > target specific function? Maybe a target specific function would be > > safer. > > No, I think just what you h

Re: [hsa 3/12] HSA libgomp plugin

2015-11-09 Thread Martin Jambor
Hi, On Thu, Nov 05, 2015 at 10:47:44PM +, Joseph Myers wrote: > This new file should have the standard libgomp copyright / license notice. > Oops, thanks for pointing this out. I am about to commit the following remedy to the branch. Thanks, Martin 2015-11-09 Martin Jambor *

Re: [hsa 1/12] Configuration and offloading-related changes

2015-11-09 Thread Martin Jambor
On Thu, Nov 05, 2015 at 10:47:15PM +, Joseph Myers wrote: > On Thu, 5 Nov 2015, Martin Jambor wrote: > > > libgomp plugin to be built. Because the plugin needs to use HSA > > run-time library, I have introduced options --with-hsa-runtime (and > > more precise --with-hsa-include and --with-hsa

Re: RFC: Experimental use of Sphinx for GCC documentation

2015-11-09 Thread Kyrill Tkachov
Hi David, On 08/11/15 13:55, David Malcolm wrote: I've been experimenting with using Sphinx [1] for GCC's documentation. You can see an HTML sample of GCC docs built with Sphinx here: https://dmalcolm.fedorapeople.org/gcc/2015-08-31/rst-experiment/gcc.html (it's a work-in-progress; i.e. there a

[PATCH 07/12] always define DBX_DEBUGGING_INFO

2015-11-09 Thread tbsaunde+gcc
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. * config/rs6000/rs6000.c (macho_branch_islands): Ad

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

2015-11-09 Thread Ramana Radhakrishnan
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 failur

[PATCH 11/12] always define HAVE_AS_LEB128

2015-11-09 Thread tbsaunde+gcc
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): Likewise. (dw2_asm_output_data_sleb128): Likewis

[PATCH 12/12] always define ENABLE_OFFLOADING

2015-11-09 Thread tbsaunde+gcc
From: Trevor Saunders gcc/cp/ChangeLog: 2015-11-09 Trevor Saunders * parser.c (cp_parser_omp_declare_target): Adjust. gcc/ChangeLog: 2015-11-09 Trevor Saunders * configure: Regenerate. * configure.ac: Always define ENABLE_OFFLOADING. * cgraph.c (cgraph_n

[PATCH 09/12] always define TARGET_PECOFF

2015-11-09 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * config/i386/i386.h (TARGET_PECOFF): Remove define. * defaults.h (TARGET_PECOFF): New default definition. * varasm.c (handle_vtv_comdat_section): Adjust. gcc/cp/ChangeLog: 2015-11-09 Trevor Saunders

[PATCH 10/12] always define EH_RETURN_HANDLER_RTX

2015-11-09 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * defaults.h (EH_RETURN_HANDLER_RTX): New default definition. * df-scan.c (df_get_exit_block_use_set): Adjust. * except.c (expand_eh_return): Likewise. --- gcc/defaults.h | 4 gcc/df-scan.c | 2 --

[PATCH 06/12] always define DWARF2_DEBUGGING_INFO

2015-11-09 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * config/arc/arc.h: Define DWARF2_DEBUGGING_INFO to 1. * config/microblaze/microblaze.h: Likewise. * defaults.h (DWARF2_DEBUGGING_INFO): New default definition. * doc/tm.texi: Regenerate.

[PATCH 04/12] always define XCOFF_DEBUGGING_INFO

2015-11-09 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * collect2.c (scan_prog_file): Remove check if XCOFF_DEBUGGING_INFO is defined. * config/rs6000/rs6000.c (macho_branch_islands): Likewise. * dbxout.c (struct dbx_file): Likewise. (default_

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

2015-11-09 Thread tbsaunde+gcc
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 wasn't compiled at all. gcc/Cha

[PATCH 08/12] always define DWARF2_LINENO_DEBUGGING_INFO

2015-11-09 Thread tbsaunde+gcc
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): Likewise. * toplev.c (process_options): Likewise. -

[PATCH 05/12] always define VMS_DEBUGGING_INFO

2015-11-09 Thread tbsaunde+gcc
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 (output_file_names): Likewise. (add_name_and_src_coords_attrib

[PATCH 02/12] remove EXTENDED_SDB_BASIC_TYPES

2015-11-09 Thread tbsaunde+gcc
From: Trevor Saunders 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): Likewise.

[PATCH 01/12] reduce conditional compilation for HARD_FRAME_POINTER_IS_ARG_POINTER

2015-11-09 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-11-09 Trevor Saunders * dbxout.c (dbxout_symbol_location): Remove #if for HARD_FRAME_POINTER_IS_ARG_POINTER. (dbxout_parms): Likewise. * dwarf2out.c (rtl_for_decl_location): Likewise. * emit-rtl.c (gen_rtx_REG)

[PATCH 00/12] misc conditional compilation work

2015-11-09 Thread tbsaunde+gcc
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 bootstrapped + regtested on x86_64-linux-gnu, and a slightly earlier version of the series

[PATCH][haifa-sched] PR rtl-optimization/68236: Exit early from autoprefetcher lookahead if not in haifa sched

2015-11-09 Thread Kyrill Tkachov
Hi all, In this PR there are tons of ICEs throughout the aarch64 testsuite when compiled with -fselective-scheduling -fselective-scheduling2 --param=sched-autopref-queue-depth=10. Admittedly that's a rare combination as we don't use selective scheduling by default on aarch64. The problem is th

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

2015-11-09 Thread Tom de Vries
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

2015-11-09 Thread Richard Sandiford
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 (aarch64_built

Re: [OpenACC] declare directive

2015-11-09 Thread James Norris
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 t

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

2015-11-09 Thread Bernd Schmidt
On 11/09/2015 02:32 PM, Robert Suchanek wrote: The results below were generated for CSiBE benchmark and the numbers in columns express bytes in format 'net (gain/loss)' to show the difference with and without the patch when -frename-registers switch is used. I'm not entirely sure what the numbe

[PATCH 5/6] Simplify rs6000_builtin_vectorized_function

2015-11-09 Thread Richard Sandiford
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/rs6000/rs6000.c (rs6000_builtin_vectorized_functi

Re: [PR66326, Cilk+] Fix FP exception occured in CilkPlus runtime

2015-11-09 Thread Thomas Schwinge
Hi! On Mon, 2 Nov 2015 20:10:06 +, "Zamyatin, Igor" wrote: > This patches fixes FP exception that comes from CilkPlus runtime. > > Bootstrapped and regtested for x86_64. > --- /dev/null > +++ b/gcc/testsuite/g++.dg/cilk-plus/CK/pr66326.cc > @@ -0,0 +1,33 @@ > +/* { dg-options "-fcilkplus"

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

2015-11-09 Thread Segher Boessenkool
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. We generate sign_extend loads for many cases where zero_extend would be pre

[PATCH 4/6] Simplify ix86_builtin_vectorized_function

2015-11-09 Thread Richard Sandiford
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/i386/i386.c (ix86_builtin_vectorized_function): R

[PATCH 3/6] Vectorize internal functions

2015-11-09 Thread Richard Sandiford
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 accordi

[PATCH 2/6] Make builtin_vectorized_function take a combined_fn

2015-11-09 Thread Richard Sandiford
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 s

Re: [PATCH 2/2] rs6000: Extend 20050603-3.c testcase to 64-bit

2015-11-09 Thread David Edelsohn
On Sun, Nov 8, 2015 at 11:33 PM, Segher Boessenkool wrote: > The testcase used to fail on 64-bit, but it was disabled there. > This patch makes it run there, and beefs up the checking of the > generated code a bit. > > Tested on powerpc64-linux *-m32,-m32/-mpowerpc64,-m64). > Is this okay for trun

[PATCH 1/6] Use IFN_SQRT in tree-vect-patterns.c

2015-11-09 Thread Richard Sandiford
In practice all targets that can vectorise sqrt define the appropriate sqrt2 optab. The only case where this isn't immediately obvious is the libmass support in rs6000.c, but Mike Meissner said that it shouldn't be exercised for sqrt. This patch therefore uses the internal function interface inst

Re: [OpenACC] declare directive

2015-11-09 Thread Jakub Jelinek
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 that (once in some copy clause, once in link), or menti

[PATCH 0/6] Automatically use vector optabs

2015-11-09 Thread Richard Sandiford
The main goal of this series is to allow functions to be vectorised simply by defining the associated optab. At the moment you can get a scalar square root instruction by defining an md pattern like sqrtdf2. But if you want to have vectorised sqrt, you need to have a target- specific C-level built

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

2015-11-09 Thread David Edelsohn
On Sun, Nov 8, 2015 at 4:36 PM, Michael Meissner wrote: > This is patch #2. It adds support for the new modulus instructions that are > being added in ISA 3.0 (power9): > > I have built this patch (along with patches #3 and #4) with a bootstrap build > on a power8 little endian system. There wer

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

2015-11-09 Thread David Edelsohn
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 development and is not ready to be >

[PATCH, 4/16] Implement -foffload-alias

2015-11-09 Thread Tom de Vries
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: [OpenACC] declare directive

2015-11-09 Thread James Norris
Jakub, This is an update of the patch from: https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00893.html. There was an unused variable in c/c-parser.c that was removed. I've included the ChangeLog as a convenience, but nothing was changed in the file. Thanks! Jim 2015-XX-XX James Norris

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

2015-11-09 Thread Segher Boessenkool
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 CTZ_DEFINED_VALUE_AT_ZERO as well. > +(define_insn "ctz2_hw" > + [(set (match_operand:GPR 0 "

Re: Add a combined_fn enum

2015-11-09 Thread Richard Sandiford
Bernd Schmidt writes: > On 11/09/2015 03:57 PM, Richard Sandiford wrote: > >> I'm not sure what you mean by "fix" though. I don't think we can change >> any of the constraints above. > > I don't see why they couldn't be relaxed. What's stopping us from > defining some builtins independently of t

[PATCH, 3/16] Ignore reduction clause on kernels directive

2015-11-09 Thread Tom de Vries
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][AArch64][cleanup] Remove uses of CONST_DOUBLE_HIGH, CONST_DOUBLE_LOW

2015-11-09 Thread Marcus Shawcroft
On 9 November 2015 at 11:34, Kyrill Tkachov wrote: > * config/aarch64/aarch64.c (aarch64_simd_valid_immediate): > Remove integer CONST_DOUBLE handling. It should never occur. OK /Marcus

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

2015-11-09 Thread Segher Boessenkool
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. > (check_effective_target_powerpc_p9vector_ok): Likewise. It's probably better not to use this

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

2015-11-09 Thread Marcus Shawcroft
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 >>> * config/aarch64/aarch64.h (TARGET_SUPPORTS_WIDE_INT): Define to

[PATCH, 2/16] Make create_parallel_loop return void

2015-11-09 Thread Tom de Vries
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][AArch64] PR target/68129: Define TARGET_SUPPORTS_WIDE_INT

2015-11-09 Thread Kyrill Tkachov
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. * config/aarch64/aarch64.c (aarch64_print_operand, CONST_DOUBLE):

[PATCH, 1/16] Insert new exit block only when needed in transform_to_exit_first_loop_alt

2015-11-09 Thread Tom de Vries
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 v3 2/2] [PR debug/67192] Further fix C loops' back-jump location

2015-11-09 Thread Andreas Krebbel
On 11/04/2015 05:18 PM, Andreas Arnez wrote: > After parsing an unconditional "while"- or "for"-loop, the C front-end > generates a backward-goto statement and implicitly sets its location to > the current input_location. But in some cases the parser peeks ahead > first, such that input_location a

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

2015-11-09 Thread Andreas Krebbel
On 11/04/2015 05:17 PM, Andreas Arnez wrote: > Since r223098 ("Implement -Wmisleading-indentation") the backward-jump > generated for a C while- or for-loop can get the wrong line number. > This is because the check for misleading indentation peeks ahead one > token, advancing input_location to aft

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

2015-11-09 Thread Tom de Vries
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 2

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

2015-11-09 Thread Marcus Shawcroft
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, CONST_DOUBLE): > Delete VOIDmode case. Assert that mode

Re: Add a combined_fn enum

2015-11-09 Thread Bernd Schmidt
On 11/09/2015 03:57 PM, Richard Sandiford wrote: I'm not sure what you mean by "fix" though. I don't think we can change any of the constraints above. I don't see why they couldn't be relaxed. What's stopping us from defining some builtins independently of the frontends, and don't all langu

Re: [PATCH PR52272]Be smart when adding iv candidates

2015-11-09 Thread Bernd Schmidt
On 11/08/2015 10:11 AM, Richard Biener wrote: On November 8, 2015 3:58:57 AM GMT+01:00, "Bin.Cheng" wrote: +inline bool +iv_common_cand_hasher::equal (const iv_common_cand *ccand1, + const iv_common_cand *ccand2) +{ + return ccand1->hash == ccand2->hash +&& op

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

2015-11-09 Thread Richard Sandiford
Michael Matz writes: > 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

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

2015-11-09 Thread Richard Biener
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 emit_library_call_value_1 does not do any argu

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

2015-11-09 Thread Michael Matz
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 op

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

2015-11-09 Thread Alan Lawrence
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 want

Re: Add a combined_fn enum

2015-11-09 Thread Richard Sandiford
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 different to be done away with entirely. built-in

Re: Add null identifiers to genmatch

2015-11-09 Thread Richard Biener
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 "null" if ALLOW_NU

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

2015-11-09 Thread Richard Biener
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 a/gcc/testsuite/gcc.target/i386/addr-space-3.c > b

Re: OpenACC Firstprivate

2015-11-09 Thread Nathan Sidwell
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.

[PATCH] Fix PR68248

2015-11-09 Thread Richard Biener
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. Index: gcc/tree-vect

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

2015-11-09 Thread Michael Matz
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: [vec-cmp, patch 4/6] Support vector mask invariants

2015-11-09 Thread Richard Biener
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); >> > + { >> > + /* Can't use VIEW_CONVERT_

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

2015-11-09 Thread Richard Biener
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 need to release auto_vec<>s at the end of scope exp

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

2015-11-09 Thread Richard Biener
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 : On Thu, Oct 8, 2015 at 4:59 PM, Ilya Enkovich wrote: > Hi, > > Th

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

2015-11-09 Thread Richard Biener
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, Oct 28, 2015 at 2:13 PM, Ilya Enkovich >> >> wrote: >> >>> Hi, >> >>> >> >>>

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

2015-11-09 Thread Richard Biener
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 finishes? Ok. Richar

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

2015-11-09 Thread Richard Biener
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 x86_64-linux-pc and regression test

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

2015-11-09 Thread Ramana Radhakrishnan
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: OpenACC Firstprivate

2015-11-09 Thread Jakub Jelinek
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: RFC: C++ delayed folding merge

2015-11-09 Thread Jason Merrill
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 h

Re: OpenACC Firstprivate

2015-11-09 Thread Nathan Sidwell
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] Use signed boolean type for boolean vectors

2015-11-09 Thread Ilya Enkovich
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, > >>> > >>> Testing boolean vector conversions I found several runtime regressi

Re: OpenACC Firstprivate

2015-11-09 Thread Nathan Sidwell
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: [vec-cmp, patch 2/6] Vectorization factor computation

2015-11-09 Thread Ilya Enkovich
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 >>> wrote: Hi, This patch handles statements with boolean result in vectorization fa

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

2015-11-09 Thread Matthew Wahab
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 Adv.SIMD instructions, sqrdmlah and sqrdmlsh. This patch adds

Re: OpenACC Firstprivate

2015-11-09 Thread Jakub Jelinek
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 reg

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

2015-11-09 Thread Ramana Radhakrishnan
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. > (neon_vld2_lane): Likewise. > (neon_vst2_lane): Lik

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

2015-11-09 Thread Christophe Lyon
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 and sqrdmlsh. This patch adds the NEON intrinsics vqrdml

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

2015-11-09 Thread Robert Suchanek
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 wi

[PATCH] 02/N Fix memory leaks in IPA

2015-11-09 Thread Martin Liška
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 00:00

<    1   2   3   >