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

On Fri, Nov 6, 2015 at 9:24 PM, Richard Biener wrote: > On Wed, Nov 4, 2015 at 11:18 AM, Bin Cheng wrote: >> Hi, >> PR52272 reported a performance regression in spec2006/410.bwaves once GCC is >> prevented from representing address of one memory object using address of >> another memory object.

Re: [PATCH] Add -fchecking

On 11/07/2015 01:47 PM, Gerald Pfeifer wrote: On Tue, 27 Oct 2015, Richard Biener wrote: This adds -fchecking as a way to enable internal consistency checks even in release builds (or disable checking with -fno-checking - up to a certain extent - with checking enabled). How (much) do we want t

[PATCH 4a/4] [ARM] PR63870 Use internal_error() for invalid lane numbers

From: Charles Baylis Charles Baylis * config/arm/neon.md (neon_vld1_lane): Use internal_error for invalid lane number. (neon_vst1_lane): Likewise. (neon_vld2_lane): Likewise. (neon_vst2_lane): Likewise. (neon_vld3_lane): Likewise. (neon

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

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): Likewise. (neon_vld3_lane): Likewise. (neon_vst3_

[PATCH 2/4] [ARM] PR63870 Mark lane indices of vldN/vstN with appropriate qualifier

From: Charles Baylis gcc/ChangeLog: Charles Baylis PR target/63870 * config/arm/arm-builtins.c: (arm_load1_qualifiers) Use qualifier_struct_load_store_lane_index. (arm_storestruct_lane_qualifiers) Likewise. * config/arm/neon.md: (neon_vld1_lane) Rever

[PATCH v3 0/4] [ARM] PR63870 vldN_lane/vstN_lane error messages

From: Charles Baylis Previous discussion: https://gcc.gnu.org/ml/gcc-patches/2015-10/msg00657.html This is a minor update to the previous patch set, fixing one coding style issue in the first patch, and adding a fourth patch for which there are two options, described below. [ARM] PR63870 Add

[PATCH 1/4] [ARM] PR63870 Add qualifiers for NEON builtins

From: Charles Baylis gcc/ChangeLog: Charles Baylis PR target/63870 * config/arm/arm-builtins.c (enum arm_type_qualifiers): New enumerator qualifier_struct_load_store_lane_index. (builtin_arg): New enumerator NEON_ARG_STRUCT_LOAD_STORE_LANE_INDEX. (arm

Re: [PATCH] Add support for ARM embedded multilibs

Hello Ramana and Tejas! The patch is originally from Terry Guo (see https://gcc.gnu.org/ml/gcc-patches/2014-05/msg00729.html). SVN commit r210320 on svn://gcc.gnu.org/svn/gcc/branches/ARM/embedded-4_9-branch . The original was using "with_multilib_list" instead of TM_MULTILIB_CONFIG. Moreover, i

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

On Tue, Oct 20, 2015 at 10:10:44PM +, Joseph Myers wrote: > > typedef struct FA5_7 { > > int i; > > char a5_7 [5][7]; > > } FA5_7; > > > > __builtin_offsetof (FA5_7, a5_7 [0][7]), // { dg-warning > > "index" } > > __builtin_offsetof (FA5_7, a5_7 [1][7]), // { dg-wa

Re: Fix ipa-polymorphic-call-info ICE

> Hi! > > On Fri, 6 Nov 2015 07:07:27 +0100, Jan Hubicka wrote: > > this patch fixes tripple thinko when in > > ipa_polymorphic_call_context::restrict_to_inner_type when dealing with an > > offset that is out of the range of the type considered. In this case > > function > > should return true

Re: [PATCH 1/9] ENABLE_CHECKING refactoring

On Wed, 21 Oct 2015, Jeff Law wrote: > I might even claim it's already helping. While we're still seeing > syntax errors in the conditionally compiled code, it doesn't feel like > we're seeing it as often as in the past. That's purely anecdotal based > on what I've seen fly by over the last co

[SPARC] Add missing final period

Applied on the mainline. 2015-11-07 Eric Botcazou * config/sparc/sparc.opt (mfix-at697f): Add final period. -- Eric BotcazouIndex: config/sparc/sparc.opt === --- config/sparc/sparc.opt (revision 229840) +++ config/sparc

[committed] gcc.dg/Wno-frame-address.c: Skip on hppa*-*-*

We need to skip on hppa since it's not possible to access arbitrary frames. Dave -- John David Anglin dave.ang...@bell.net 2015-11-07 John David Anglin * gcc.dg/Wno-frame-address.c: Skip on hppa*-*-*. Index: gcc.dg/Wno-frame-address.c ==

Re: [PATCH] Add -fchecking

On Tue, 27 Oct 2015, Richard Biener wrote: > This adds -fchecking as a way to enable internal consistency checks > even in release builds (or disable checking with -fno-checking - up to > a certain extent - with checking enabled). How (much) do we want to advertize this? Gerald

[PATCH] PR fortran/68244 -- Check for NULL() in an array spec.

NULL() can only appear in a few situations. It cannot be part of an array spec. See testcase for example. OK to commit? 2015-11-07 Steven G. Kargl PR fortran/68224 * array.c (match_array_element_spec): Check of invalid NULL(). While here, fix nearby comments. 2015-11

Re: Combined constructs' clause splitting

On 11/07/2015 03:45 AM, Thomas Schwinge wrote: > Hi! > > On Fri, 6 Nov 2015 15:31:23 -0800, Cesar Philippidis > wrote: >> I've applied this patch to gomp-4_0-branch which backports most of my >> front end changes from trunk. Note that I found a regression while >> testing, which is also present

Re: [gomp4] backport trunk FE changes

On 11/07/2015 04:30 AM, Thomas Schwinge wrote: > Hi! > > On Fri, 6 Nov 2015 15:31:23 -0800, Cesar Philippidis > wrote: >> I've applied this patch to gomp-4_0-branch which backports most of my >> front end changes from trunk. > >> --- a/gcc/cp/pt.c >> +++ b/gcc/cp/pt.c >> @@ -14398,7 +14398,6 @@

Re: [Patch, fortran] PR68196 [4.9/5/6 Regression] ICE on function result with procedure pointer component

On Wed, Nov 04, 2015 at 04:03:10PM +0100, Paul Richard Thomas wrote: > > 2015-11-04 Paul Thomas > > PR fortran/68196 > * class.c (has_finalizer_component): Prevent infinite recursion > through this function if the derived type and that of its > component are the same. > * t

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

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 still working on. I am sending this out f

Re: Add null identifiers to genmatch

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_NULL. */ > > id_base * > -get_operator (const char *id) > +get_operator (const char *id, bool allow_null = false) >

[PATCH] Complete cxa_atexit for AIX

IBM xlC++ compiler provides its own implementation of atexit() to provide correct interaction between atexit() and destructors. GCC needs to provide the same through libgcc. I previously added partial support based on the implementation in GLIBC (copied with FSF permission) and confirmed that des

Re: [Fortran, patch, pr68218, v1] ALLOCATE with size given by a module function

Dear Andre, OK for trunk. I understand that you have investigated the issue(s) reported to you by Dominique and can find no sign of them. Thanks Paul On 5 November 2015 at 15:29, Andre Vehreschild wrote: > Hi all, > > attached is a rather trivial patch to prevent multiple evaluations of a > f

[gomp4] Merge trunk r229809 (2015-11-05) into gomp-4_0-branch

Hi! Committed to gomp-4_0-branch in r229929: commit f782e15f314aa57eb7bca3bfdea54fba6c48e929 Merge: eb7d11e e103794 Author: tschwinge Date: Sat Nov 7 13:42:07 2015 + svn merge -r 229770:229809 svn+ssh://gcc.gnu.org/svn/gcc/trunk git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/

OpenACC Firstprivate

Jakub, this patch implements firstprivate support for openacc. This is pretty straight forwards -- they're just regular auto variables, but with an initialization value from the host. The gimplify.c implementation is somewhat different to gomp4 branch, as I've added new bits to enum omp_regi

Re: [sh] Add flag_unsafe_math_optimizations to sincossf3

On Sat, 2015-11-07 at 13:27 +, Richard Sandiford wrote: > builtins.c uses the following code to guard expansions involving > optabs: > > CASE_FLT_FN (BUILT_IN_EXP): > CASE_FLT_FN (BUILT_IN_EXP10): > CASE_FLT_FN (BUILT_IN_POW10): > CASE_FLT_FN (BUILT_IN_EXP2): > CASE_FLT_FN

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

For -fmath-errno, builtins.c currently expands calls to sqrt to: y = sqrt_optab (x); if (y != y) [ sqrt (x); or errno = EDOM; ] The drawbacks of this are: - the call to sqrt is protected by the result of the optab rather than the input. It would be better to check !(x >= 0), lik

Short-cut generation of simple built-in functions

This patch short-circuits the builtins.c expansion code for a particular gimple call if: - the function has an associated internal function - the target implements that internal function - the call has no side effects This allows a later patch to remove the builtins.c code, once calls with side e

[sh] Add flag_unsafe_math_optimizations to sincossf3

builtins.c uses the following code to guard expansions involving optabs: CASE_FLT_FN (BUILT_IN_EXP): CASE_FLT_FN (BUILT_IN_EXP10): CASE_FLT_FN (BUILT_IN_POW10): CASE_FLT_FN (BUILT_IN_EXP2): CASE_FLT_FN (BUILT_IN_EXPM1): CASE_FLT_FN (BUILT_IN_LOGB): CASE_FLT_FN (BUILT_IN

Replace match.pd DEFINE_MATH_FNs with auto-generated lists

This patch autogenerates the operator lists for maths functions like SQRT, adding an additional entry for internal functions. E.g.: (define_operator_list SQRT BUILT_IN_SQRTF BUILT_IN_SQRT BUILT_IN_SQRTL IFN_SQRT) and: (define_operator_list CABS BUI

Add null identifiers to genmatch

This patch adds a null identifier that can never match anything and can never be generated. It is only valid in operator lists and fors. Later patches will add uses of it. The idea is to allow operator lists for maths functions that have four entries: - float built-in - double built-in - long do

Extend mathfn_built_in to handle combined_fn

This patch extends mathfn_built_in to handle combined_fn, but keeps the old built_in_function interface around since it's a common case. Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi. OK to install? Thanks, Richard gcc/ * builtins.h (mathfn_built_in): Add a variant

Use combined_fn in tree-vect-patterns.c

Another patch to extend uses of built_in_function to combined_fn, this time in tree-vect-patterns.c. The old code didn't handle the long double pow variants, but I think that's because noone had a target that would benefit rather than because the code would mishandle them. Tested on x86_64-linux-

Use combined_fn in tree-ssa-math-opts.c

Another patch to extend uses of built_in_function to combined_fn, this time in tree-ssa-math-opts.c. Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi. OK to install? Thanks, Richard gcc/ * tree-ssa-math-opts.c: Include case-cfn-macros.h. (execute_cse_sincos_1)

Use combined_fn in tree-ssa-reassoc.c

Another patch to extend uses of built_in_function to combined_fn, this time in tree-ssa-reassoc.c. Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi. OK to install? Thanks, Richard gcc/ * tree-ssa-reassoc.c: Include case-cfn-macros.h. (stmt_is_power_of_op): Use

Use combined_fn in tree-vrp.c

Another patch to extend uses of built_in_function to combined_fn, this time in tree-vrp.c. Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi. OK to install? Thanks, Richard gcc/ * tree-vrp.c: Include case-cfn-macros.h. (extract_range_basic): Switch on combined_

Make more use of combined_fn

This patch generalises fold-const.[hc] routines to use combined_fn instead of built_in_function. It also updates gimple-ssa-backprop,c since the update is simple and it avoids churn on the call to negate_mathfn_p. Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi. OK to install?

Extend fold_const_call to combined_fn

This patch extends fold_const_call so that it can handle internal as well as built-in functions. Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi. OK to install? Thanks, Richard gcc/ * fold-const-call.h (fold_const_call): Replace built_in_function arguments wi

Add gencfn-macros.c

This patch automatically generates case macros such as: CASE_CFN_SQRT for each {F,,L} floating-point built-in function and each {,L,LL,IMAX} integer built-in function. The macros match the same built-in functions as CASE_FLT_FN and CASE_INT_FN but in addition include the associated internal

Add internal bitcount functions

This patch adds internal function equivalents of all the INT_FN functions. Unlike the math functions, these functions never set errno and the internal functions should be exactly equivalent to the built-in ones. The reason for defining the internal functions is so that we can extend the functional

Add internal math functions

This patch adds internal functions for simple FLT_FN built-in functions, in cases where an associated optab already exists. Unlike some of the built-in functions, these internal functions never set errno. LDEXP is an odd-one out in that its second operand is an integer. All the others operate on

Add basic support for direct_optab internal functions

This patch adds a concept of internal functions that map directly to an optab (here called "direct internal functions"). The function can only be used if the associated optab can be used. We currently have four functions like that: - LOAD_LANES - STORE_LANES - MASK_LOAD - MASK_STORE so the patc

Add a combined_fn enum

I'm working on a patch series that needs to be able to treat built-in functions and internal functions in a similar way. This patch adds a new enum, combined_fn, that combines the two together. It also adds utility functions for seeing which combined_fn (if any) is called by a given CALL_EXPR or

Re: improved RTL-level if conversion using scratchpads [half-hammock edition]

On November 6, 2015 5:27:44 PM GMT+01:00, Bernd Schmidt wrote: >On 11/06/2015 04:52 PM, Sebastian Pop wrote: > >>> opinion). If you want a half-finished redzone allocator, I can send >you a >>> patch. >> >> Yes please. Let's get it work. > >Here you go. This is incomplete and does not compile, b

Combined constructs' clause splitting (was: [gomp4] backport trunk FE changes)

Hi! On Fri, 6 Nov 2015 15:31:23 -0800, Cesar Philippidis wrote: > I've applied this patch to gomp-4_0-branch which backports most of my > front end changes from trunk. Note that I found a regression while > testing, which is also present in trunk. It looks like > kernels-acc-loop-reduction.c is

[RFC] [PATCH V2]: RE: [RFC] [Patch] Relax tree-if-conv.c trap assumptions.

Hi Richard, > -Original Message- > From: Richard Biener [mailto:richard.guent...@gmail.com] > Sent: Friday, October 30, 2015 5:00 PM > To: Kumar, Venkataramanan > Cc: Andrew Pinski; gcc-patches@gcc.gnu.org > Subject: Re: [RFC] [Patch] Relax tree-if-conv.c trap assumptions. > > On Fri, Oct

Re: [PATCH 1/7] New obstack_next_free is not an lvalue

Alan Modra writes: > diff --git a/gcc/gensupport.c b/gcc/gensupport.c > index 0480e17..484ead2 100644 > --- a/gcc/gensupport.c > +++ b/gcc/gensupport.c > @@ -2253,7 +2253,7 @@ htab_eq_string (const void *s1, const void *s2) > and a permanent heap copy of STR is created. */ > > static void

Re: [gomp4] OpenACC reduction tests

Hi! On Wed, 23 Sep 2015 09:56:44 +0200, I wrote: > On Fri, 18 Sep 2015 10:11:25 +0200, I wrote: > > On Fri, 17 Jul 2015 11:13:59 -0700, Cesar Philippidis > > wrote: > > > This patch updates the libgomp OpenACC reduction test cases [...] > > Given the following > > -Wuninitialized/-Wmaybe-uninit

[gomp4, committed] Revert "Add counter inits to zero_iter_bb in expand_omp_for_init_counts"

Hi, this patch reverts "Add counter inits to zero_iter_bb in expand_omp_for_init_counts". We no longer split off the kernels region in ssa-mode, so there's no need for this patch anymore. Committed to gomp-4_0-branch. Thanks, - Tom Revert "Add counter inits to zero_iter_bb in expand_omp_for_

[gomp4, committed] Remove no_overflow_tree_code

Hi, this patch removes dead code from gomp-4_0-branch. Committed to gomp-4_0-branch. Thanks, - Tom Remove no_overflow_tree_code 2015-11-07 Tom de Vries * tree.c (no_overflow_tree_code): Remove. * tree.h (no_overflow_tree_code): Remove. --- gcc/tree.c | 24 gcc/t

[gomp4, committed] Make formatting resemble trunk in f95-lang.c

Hi, this patch removes formatting differences with trunk. Committed to gomp-4_0-branch. Thanks, - Tom Make formatting resemble trunk in f95-lang.c 2015-11-07 Tom de Vries * f95-lang.c: Make formatting resemble trunk. --- gcc/fortran/f95-lang.c | 1 + 1 file changed, 1 insertion(+) diff

[gomp4, committed] Cleanup formatting of DEF_GOACC_BUILTINs

Hi, this patch removes formatting differences with trunk. Committed to gomp-4_0-branch. Thanks, - Tom >From a77fd266102498a909886cecde1b57adf9350d90 Mon Sep 17 00:00:00 2001 From: Tom de Vries Date: Fri, 6 Nov 2015 22:11:08 +0100 Subject: [PATCH 2/4] Cleanup formatting of DEF_GOACC_BUILTINs 2

[gomp4, committed] Undo cgraph_node::release_body workaround

Hi, this patch removes a workaround that's no longer needed, now that we split off the kernels region at the first omp-expand pass. Committed to gomp-4_0-branch. Thanks, - Tom >From 5e9a609006b45c51598a3d52d5ab55b72a186f67 Mon Sep 17 00:00:00 2001 From: Tom de Vries Date: Fri, 6 Nov 2015 22:

Re: [PATCH] i386: Use the STC bb-reorder algorithm at -Os (PR67864)

On Sat, Nov 7, 2015 at 3:34 AM, Segher Boessenkool wrote: > Adding x86 maintainer, ping? > > On Fri, Oct 16, 2015 at 05:53:41AM -0700, Segher Boessenkool wrote: >> For x86, STC still gives better results for optimise-for-size than >> "simple" does. So use STC at -Os as well. >> >> Is this okay fo

Re: [AArch64] Fix vqtb[lx][234] on big-endian

On Fri, Nov 06, 2015 at 09:37:17PM +0100, Christophe Lyon wrote: > On 6 November 2015 at 18:03, James Greenhalgh > wrote: > > On Fri, Nov 06, 2015 at 02:49:38PM +0100, Christophe Lyon wrote: > >> Hi, > >> > >> As mentioned by James a few weeks ago, the vqtbl[lx][234] intrinsics > >> are failing o

***ping*** [Patch, fortran] PR68196 [4.9/5/6 Regression] ICE on function result with procedure pointer component

***ping*** On 4 November 2015 at 16:03, Paul Richard Thomas wrote: > Dear All, > > The patch for these PRs is fully explained by the the comments and/or > changelogs. PR66465 has no connection with PR68196, other than Damian > asking if it is connected! > > Bootstrapped and regtested on x86_64/FC

[PATCH 7/7] Configury changes for obstack optimization

Provides defines used to determine whether glibc obstacks are compatible. Generally speaking, 32-bit targets won't need to use obstack.o from libiberty if glibc is used, while 64-bit targets will, until glibc gets the new obstack code. * configure.ac: Check size of size_t. * confi

[PATCH 6/7] Silence obstack.c -Wc++compat warning

Fixes warning: request for implicit conversion from ‘void *’ to ‘struct _obstack_chunk *’ not permitted in C++ [-Wc++-compat] I moved the assignment to h->chunk to fix an overlong line, then decided it would be better after the alloc failure check just to do things the same way as in _obstack_new

[PATCH 5/7] Modify obstack.[hc] to avoid having to include other gnulib files

Using the standard gnulib obstack source requires importing quite a lot of other files from gnulib, and requires build changes. If one did want to use gnulib obstack directly, then it would need to go in a sub-directory and after ".../gnulib-tool --import obstack" we'd have the following: ./lib:

[PATCH 4/7] Copy gnulib obstack files

This copies obstack.[ch] from gnulib, and updates the docs. The next patch should be applied if someone repeats the import at a later date. include/ PR gdb/17133 * obstack.h: Import current gnulib file. libiberty/ PR gdb/17133 * obstack.c: Import current gnulib fil

[PATCH 3/7] Update libsanitizer obstack interceptors

New obstack uses sensible types, size_t instead of int for length params. Since libsanitizer does not use prototypes from obstack.h to call the real functions, it's necessary to update the libsanitizer function declarations emitted by the INTERCEPTOR macro. As per the comment added to configure.a

[PATCH 2/7] Correct libvtv obstack use

Fixes a compile error with both old and new obstacks due to obstack_chunk_free having the wrong signature. Also, setting chunk size and alignment before obstack_init is pointless since they are overwritten. * vtv_malloc.cc (obstack_chunk_free): Correct param type. (__vtv_malloc_in

[PATCH 1/7] New obstack_next_free is not an lvalue

New obstack.h casts obstack_next_free to (void *), resulting in it being a non-lvalue, and warnings on pointer arithmetic. gcc/ * gensupport.c (add_mnemonic_string): Make len param a size_t. (gen_mnemonic_setattr): Make "size" var a size_t. Use obstack_blank_fast to shrink

[PATCH 0/7] 64-bit obstack support in libiberty

This patchset imports new obstack support to libiberty, to better support 64-bit systems, and fix an old gdb bug. Most of the necessary changes outside of libiberty were committed October last year, but a few more incompatibilities have crept in since then. The first three patches fix these probl