Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1

2014-11-13 Thread Thomas Schwinge
Hi! On Wed, 12 Nov 2014 16:58:54 +0100, I wrote: > Hi Casar! > > Can you please help me trying to understand the following change, that > you've done a long time ago: > > On Tue, 8 Jul 2014 07:28:24 -0700, Cesar Philippidis > wrote: > > On 07/07/2014 02:55 AM, Thomas Schwinge wrote: > > > > >

Re: [gomp4] fix omp_clause_code_name

2014-11-13 Thread Thomas Schwinge
Hi Cesar! On Wed, 12 Nov 2014 21:34:49 -0800, Cesar Philippidis wrote: > debug_tree was throwing an ICE when I was debugging scan_sharing_clauses > in omp-low.c. The problem turned out to be a missing comma. I've applied > this fix to gomp-4_0-branch. > --- gcc/tree.c(revision 217459) >

Re: [PATCH, MPX runtime 1/2] Integrate MPX runtime library

2014-11-13 Thread Ilya Enkovich
2014-11-13 2:03 GMT+03:00 Joseph Myers : > On Thu, 13 Nov 2014, Ilya Enkovich wrote: > >> It's hard to decide which of runtime functionality should be >> considered as basic and how it should be used. We may say that the >> only basic thing is hardware enabling which is enable_mpx and stop >> here

[PATCH] Propagate nonfreeing_call_p using ipa-pure-const (take 2)

2014-11-13 Thread Jakub Jelinek
Hi! On Thu, Nov 13, 2014 at 12:11:08AM +0100, Jan Hubicka wrote: > Actually I think you want to do this for can_throw, too. > We probably do not have throwing internal calls, but it is better to be safe. I'll leave that change to you ;), as I said in my last mail, it isn't immediately clear to m

[PATCH] Fix PR61559

2014-11-13 Thread Richard Biener
The following fixes PR61559 by implementing the bswap cancellation patters required for gcc.dg/builtin-bswap-8.c in match.pd. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2014-12-13 Richard Biener PR middle-end/61559 * match.pd: Implement bswap patt

Re: [PATCH 2/n] OpenMP 4.0 offloading infrastructure: LTO streaming

2014-11-13 Thread Richard Biener
On Wed, 12 Nov 2014, Ilya Verbin wrote: > On 12 Nov 15:34, Richard Biener wrote: > > > > Seems like Vlad introduced the conflicting type with > > > Okay, we're going to test our changes with patch in the bottom applied > > > both to kyukhin/gomp4-offload and trunk. > > > > > > * gcc/lra-live

Re: [PATCH] Introduce TYPE_OVERFLOW_SANITIZED

2014-11-13 Thread Richard Biener
On Wed, 12 Nov 2014, Marek Polacek wrote: > This patch introduces TYPE_OVERFLOW_SANITIZED predicate to > consolidate a common check. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? Hmm, I'd have expected to test !TYPE_OVERFLOW_SANITIZED (), that is, have the predicate inverted. At lea

RE: [PATCH, RFC, C] Add -fno-float to forbid floating point data types

2014-11-13 Thread Thomas Preud'homme
> From: Joseph Myers [mailto:jos...@codesourcery.com] > Sent: Wednesday, November 12, 2014 10:11 PM > > > > This patch modifies the C parser to give an error if: > > - any variable or function parameter is declared with a float type or > > a type containing a float (prototype are ignored) > > Bu

RL78: Add support for assembler G10 multiply/divide functions to libgcc

2014-11-13 Thread Nick Clifton
Hi Guys, I am applying the patch below to add support for G10 versions of the RL78 assembler multiply and divide functions to the libgcc library. Cheers Nick libgcc/ChangeLog 2014-11-13 Nick Clifton * config/rl78/divmodhi.S: Add support for the G10 architecture. Use STA

[PATCH][19/n] Merge from match-and-simplify, conditional patterns

2014-11-13 Thread Richard Biener
This merges simplifications of conditionals from tree-ssa-forwprop.c. Slight complication arises from having to also implement 0 ? A : B because otherwise X ? A : A may trigger which may trigger an overflow warning during bootstrap if A and A are not equal but 0 and 0(OVF) but we selected the "wr

Re: Run pass_expand_omp_ssa after pass_paralellize_loops

2014-11-13 Thread Richard Biener
On Wed, 12 Nov 2014, Tom de Vries wrote: > On 12-11-14 15:17, Richard Biener wrote: > > On Wed, 12 Nov 2014, Tom de Vries wrote: > > > > > [ moved from gcc@ to gcc-patches@ ] > > > [ subject was: Re: [gomp4] openacc kernels directive support ] > > > On 30-09-14 15:37, Tom de Vries wrote: > > > >

Re: [PATCH 0/4][Vectorizer] Reductions: replace VEC_RSHIFT_EXPR with VEC_PERM_EXPR

2014-11-13 Thread Richard Biener
On Wed, 12 Nov 2014, Alan Lawrence wrote: > In response to https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01803.html, this > series removes the VEC_RSHIFT_EXPR, instead using a VEC_PERM_EXPR (with a > second argument full of constant zeroes) to represent the shift. > > I've kept the use of vec_shr

[PING][RFC PATCH, AARCH64] Add support for -mlong-calls option

2014-11-13 Thread Yangfei (Felix)
Ping? I hope this patch can catch up with stage 1 of GCC-5.0. Thanks. > > Hi Felix, > > > > Sorry for the delay responding, I've been out of the office recently > > and I'm only just catching up on a backlog of GCC related emails. > > > > I'm in two minds about this; I can potentially see th

Re: libsanitizer merge from upstream r221802

2014-11-13 Thread Jakub Jelinek
On Wed, Nov 12, 2014 at 05:35:48PM -0800, Konstantin Serebryany wrote: > Here is one more merge of libsanitizer (last one was in Sept). > > Tested on x86_64 Ubuntu 14.04 like this: > rm -rf */{*/,}libsanitizer && make -j 50 > make -j 40 -C gcc check-g{cc,++} > RUNTESTFLAGS='--target_board=unix\{-m

Re: [PATCH 2/5] OpenACC 2.0 support for libgomp - temporarily work around missing __builtin_acc_on_device (repost)

2014-11-13 Thread Jakub Jelinek
On Tue, Nov 11, 2014 at 01:53:49PM +, Julian Brown wrote: > On Tue, 23 Sep 2014 19:19:55 +0100 > Julian Brown wrote: > > > The patches implementing __builtin_acc_on_device are still in > > processing. For the time being this patch removes the dependency on > > that builtin in the OpenACC runt

[PATCH] DW_AT_APPLE_* DWARF extensions.

2014-11-13 Thread Andrew Burgess
Some time ago there was an attempt to add the DWARF DW_AT_APPLE_* extensions to the file include/dwarf2.def. The original patch email is here: https://sourceware.org/ml/binutils/2012-09/msg00282.html the "patch committed" mail is here: https://sourceware.org/ml/binutils/2012-10/msg00424.html w

Re: [PATCH 3/5] OpenACC 2.0 support for libgomp - outline documentation (repost)

2014-11-13 Thread Jakub Jelinek
On Tue, Nov 11, 2014 at 01:54:08PM +, Julian Brown wrote: > From 1f17beb70b5607d1884fad1cb4734857f0e7846f Mon Sep 17 00:00:00 2001 > From: Julian Brown > Date: Mon, 22 Sep 2014 02:45:29 -0700 > Subject: [PATCH 3/5] OpenACC documentation. > > -xx-xx Thomas Schwinge > James Nor

[PATCH][AArch64] Use std::swap instead of manually swapping

2014-11-13 Thread Kyrill Tkachov
Hi all, Following the trend in i386 and alpha, this patch uses std::swap to perform swapping of values in the aarch64 backend instead of declaring temporaries. Tested and bootstrapped on aarch64-linux. Ok for trunk? Thanks, Kyrill 2014-11-13 Kyrylo Tkachov * config/aarch64/aarch64.

[PATCH][ARM] Use std::swap instead of manually swapping

2014-11-13 Thread Kyrill Tkachov
Hi all, Following the trend in i386 and alpha, this patch uses std::swap to perform swapping of values in the arm backend instead of declaring temporaries. Tested and bootstrapped on arm-none-linux-gnueabihf. Ok for trunk? Thanks, Kyrill 2014-11-13 Kyrylo Tkachov * config/arm/arm.md

Unreviewed libcc1 patch

2014-11-13 Thread Rainer Orth
The following patch [libcc1, build] Enable libcc1 on Solaris https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00079.html has remained unreviewed for more than a week. In the absence of libcc1 maintainer, it needs a global reviewer. Thanks. Rainer -- --

Re: [PATCH] DW_AT_APPLE_* DWARF extensions.

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 10:35:28AM +0100, Andrew Burgess wrote: > Am I correct to think that the include/* files are owned by GCC, and > so the mistake here was not propagating the change to the GCC > repository? Yes. > 2014-11-13 Shinichiro Hamaji > > * dwarf2.h (DW_AT_APPLE_optimized,

Re: Unreviewed libcc1 patch

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 10:44:56AM +0100, Rainer Orth wrote: > The following patch > > [libcc1, build] Enable libcc1 on Solaris > https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00079.html > > has remained unreviewed for more than a week. In the absence of libcc1 > maintainer, it nee

Re: [PATCH 3/5] OpenACC 2.0 support for libgomp - outline documentation (repost)

2014-11-13 Thread Tobias Burnus
Jakub Jelinek wrote: > > -* libgomp: (libgomp).GNU OpenMP runtime library > > +* libgomp: (libgomp).GNU OpenACC and OpenMP runtime > > library > > @end direntry > > See Dave Malcolm's patch, please integrate it into your patchset. Namely, https://gcc.gnu.o

New patch: [AArch64] [BE] [1/2] Make large opaque integer modes endianness-safe.

2014-11-13 Thread David Sherwood
Hi All, I have successfully rebased this and tested in conjunction with a patch from Alan Hayward ([AArch64] [BE] Fix vector load/stores to not use ld1/st1), who should be submitting a new version shortly. Built and tested on: aarch64-none-elf aarch64_be-none-elf x86_64-linux-gnu Regards, David

Re: [PATCH] Introduce TYPE_OVERFLOW_SANITIZED

2014-11-13 Thread Marek Polacek
On Thu, Nov 13, 2014 at 09:45:50AM +0100, Richard Biener wrote: > Hmm, I'd have expected to test !TYPE_OVERFLOW_SANITIZED (), > that is, have the predicate inverted. At least that's how > I understand the name - an overflow is to be preserved for > sanitization. I guess that macro name is a litt

Re: Unreviewed libcc1 patch

2014-11-13 Thread Richard Biener
On Thu, 13 Nov 2014, Rainer Orth wrote: > The following patch > > [libcc1, build] Enable libcc1 on Solaris > https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00079.html > > has remained unreviewed for more than a week. In the absence of libcc1 > maintainer, it needs a global reviewer

Re: [PATCH 4/5] OpenACC 2.0 support for libgomp - new tests (repost)

2014-11-13 Thread Jakub Jelinek
On Tue, Nov 11, 2014 at 01:54:42PM +, Julian Brown wrote: > @@ -169,6 +192,57 @@ proc libgomp_init { args } { > > # Disable color diagnostics > lappend ALWAYS_CFLAGS "additional_flags=-fdiagnostics-color=never" > + > +# TODO. Evil hack. DejaGnu doesn't have a mechanism for set

Re: [PATCH 5/5] OpenACC 2.0 support for libgomp - temporary test harness tweaks

2014-11-13 Thread Jakub Jelinek
On Tue, Nov 11, 2014 at 01:55:18PM +, Julian Brown wrote: > Hi, > > As mentioned in the previous mail in this series, testing the OpenACC > runtime support in libgomp is going to be awkward until the associated > middle-end pieces are ready. This stop-gap patch helps to allow tests > (that don

Re: [PATCH] Introduce TYPE_OVERFLOW_SANITIZED

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 11:11:41AM +0100, Marek Polacek wrote: > On Thu, Nov 13, 2014 at 09:45:50AM +0100, Richard Biener wrote: > > Hmm, I'd have expected to test !TYPE_OVERFLOW_SANITIZED (), > > that is, have the predicate inverted. At least that's how > > I understand the name - an overflow is

Re: [PATCH] Add a way to mark regions of code which assume that the GC won't run

2014-11-13 Thread Richard Biener
On Wed, Nov 12, 2014 at 11:04 PM, David Malcolm wrote: > On Wed, 2014-11-12 at 13:16 -0500, David Malcolm wrote: >> We make assumptions in the codebase about when the GC can run, and when >> it can't (e.g. within numerous passes) but these aren't captured in a way >> that's verifiable. >> >> This

[PATCH, libgfortran] PR 60324 Unbounded stack allocations in libgfortran

2014-11-13 Thread Janne Blomqvist
Hi, in the spirit of PR 60324 and 61035, here's a patch that gets rid of the remaining potentially unbounded stack allocations in libgfortran. All uses of __builtin_alloca() and VLA's are replaced either straight with heap allocated memory, or with a fixed size stack buffer, potentially switching

[PATCH][AArch64] Remove/merge redundant iterators

2014-11-13 Thread Alan Lawrence
Hi, gcc/config/aarch64/iterators.md contains numerous duplicates - not always obvious as they are not always sorted the same. Sometimes, one copy is used is aarch64-simd-builtins.def and another in aarch64-simd.md; othertimes there is no obvious pattern ;). This patch just removes all the du

[PING ^ 2][PATCH, AArch64] Add doloop_end pattern for -fmodulo-sched

2014-11-13 Thread Yangfei (Felix)
> On 11/12/2014 11:01 AM, Yangfei (Felix) wrote: > > +(define_expand "doloop_end" > >> + [(use (match_operand 0 "" "")) ; loop pseudo > >> + (use (match_operand 1 "" ""))] ; label > >> + "" > >> + " > >> +{ > > Drop the "" surrounding the { }. > > > r~ Hello, I updated the pa

Re: [gimple-classes, committed 4/6] tree-ssa-tail-merge.c: Use gassign

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 2:41 AM, David Malcolm wrote: > On Tue, 2014-11-11 at 11:43 +0100, Richard Biener wrote: >> On Tue, Nov 11, 2014 at 8:26 AM, Jakub Jelinek wrote: >> > On Mon, Nov 10, 2014 at 05:27:50PM -0500, David Malcolm wrote: >> >> On Sat, 2014-11-08 at 14:56 +0100, Jakub Jelinek wrot

[C++ Patch] Add maybe_constant_folded_value

2014-11-13 Thread Paolo Carlini
Hi, shall we do something like the below? Safety checked x86_64-linux. Thanks, Paolo. /// 2014-11-13 Paolo Carlini * constexpr.c (maybe_constant_folded_value): Add. * cp-tree.h (maybe_constant_folded_value): Declare it. * call.c (null_ptr_cst_p): Use

Re: LTO streaming of TARGET_OPTIMIZE_NODE

2014-11-13 Thread Richard Biener
On Thu, 13 Nov 2014, Jan Hubicka wrote: > Hi, > this patch adds infrastructure for proper streaming and merging of > TREE_TARGET_OPTION. The catch is that TREE_TARGET_OPTION is autogenerated > structure. For x86_64 it looks as follows: > > /* Structure to save/restore selected target specific o

Re: [gofrontend-dev] [PATCH 4/4] Gccgo port to s390[x] -- part II

2014-11-13 Thread Dominik Vogt
On Fri, Nov 07, 2014 at 08:24:15AM -0800, Ian Taylor wrote: > On Fri, Nov 7, 2014 at 12:51 AM, Dominik Vogt wrote: > > On Thu, Nov 06, 2014 at 09:06:18AM -0800, Ian Taylor wrote: > >> On Thu, Nov 6, 2014 at 4:04 AM, Dominik Vogt > >> wrote: > >> > On Tue, Nov 04, 2014 at 08:16:51PM -0800, Ian Ta

Re: [gimple-classes, committed 4/6] tree-ssa-tail-merge.c: Use gassign

2014-11-13 Thread Jonathan Wakely
On 13 November 2014 10:45, Richard Biener wrote: > > Hmm. > > struct assign; > struct base { > operator assign *() const { return (assign *)this; } > }; > struct assign : base { > }; > > void foo (assign *); > void bar (base *b) > { > foo (b); > } > > doesn't work, but > > void bar (base &b) >

Re: [PATCH] Propagate nonfreeing_call_p using ipa-pure-const (take 2)

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 9:39 AM, Jakub Jelinek wrote: > Hi! > > On Thu, Nov 13, 2014 at 12:11:08AM +0100, Jan Hubicka wrote: > >> Actually I think you want to do this for can_throw, too. >> We probably do not have throwing internal calls, but it is better to be safe. > > I'll leave that change to

Re: [PATCH] PR debug/38757 gcc does not emit DW_LANG_C99

2014-11-13 Thread Richard Biener
On Wed, Nov 12, 2014 at 5:31 PM, Jakub Jelinek wrote: > On Wed, Nov 12, 2014 at 04:46:39PM +0100, Mark Wielaard wrote: >> I don't mind changing the language hook to return the actual DW_LANG_* >> constant directly, that seems like a nice cleanup. >> >> But I don't think we can just replace TRANSLA

Re: PATCH: PR tree-optimization/63828: [5 Regression] g++.dg/ipa/devirt-47.C fails for x32

2014-11-13 Thread Richard Biener
On Wed, Nov 12, 2014 at 6:18 PM, H.J. Lu wrote: > Hi, > > possible_placement_new checks if the structure is at least big enough to > hold the VPTR pointer. But it uses BITS_PER_WORD for pointer size, > which is incorrect for many ILP32 targets. This patch replaces it with > POINTER_SIZE. Tested

Re: [PATCH 1/4][Vectorizer] Split vect_gen_perm_mask into _checked and _any variants

2014-11-13 Thread Richard Biener
On Wed, Nov 12, 2014 at 6:52 PM, Alan Lawrence wrote: > This is a preliminary to patch 2, which wants functionality equivalent to > vect_gen_perm_mask (converting a char* to an RTL const_vector) but without > the check of can_vec_perm_p. > > All existing calls to vect_gen_perm_mask barring that in

Re: [PATCH] PR debug/38757 gcc does not emit DW_LANG_C99

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 12:08:01PM +0100, Richard Biener wrote: > Btw, I have nothing against changing TRANSLATION_UNIT_LANGUAGE > to DW_LANG_* directly - we only use it for choosing the language > of the dwarf unit. This would of course mean to change its initialization > from > > TRANSLATION_

Re: [PATCH 2/4][Vectorizer] Use a VEC_PERM_EXPR instead of VEC_RSHIFT_EXPR; expand appropriate VEC_PERM_EXPRs using vec_shr_optab

2014-11-13 Thread Richard Biener
On Wed, Nov 12, 2014 at 6:53 PM, Alan Lawrence wrote: > This makes the vectorizer use VEC_PERM_EXPRs when doing reductions via > shifts, rather than VEC_RSHIFT_EXPR. > > VEC_RSHIFT_EXPR presently has an endianness-dependent meaning (paralleling > vec_shr_optab). While the overall destination of th

Re: [PATCH 3/4] Remove VEC_RSHIFT_EXPR tree code, now unused

2014-11-13 Thread Richard Biener
On Wed, Nov 12, 2014 at 6:55 PM, Alan Lawrence wrote: > Tested (with patches 1+2): > > Bootstrap + check-gcc on x64-none-linux-gnu > > cross-tested check-gcc on aarch64-none-elf and aarch64_be-none-elf as these > platforms stand (i.e. without vec_shr_optab). > > also cross-tested check-gcc on aarc

Re: [PATCH 4/4][Vectorizer]Make reductions-via-shifts and vec_shr_optab endianness-neutral

2014-11-13 Thread Richard Biener
On Wed, Nov 12, 2014 at 6:56 PM, Alan Lawrence wrote: > This redefines vec_shr optab to be the same (in terms of gcc vectors) > regardless of target endianness. The vectorizer uses this to do reductions > via shifts, so also change the vectorizer to shift things always the same > way (from the mid

Re: [PATCH, libgfortran] PR 60324 Unbounded stack allocations in libgfortran

2014-11-13 Thread FX
> Regtested on x86_64-unknown-linux-gnu, Ok for trunk? OK two me, with three comments: >* intrinsics/chmod.c (chmod_internal): New function, move logic >here. >(chmod_func): Call chmod_internal. Not sure what’s the need / benefit from this, given the function is only called once. >

Fix inaccuracies in doc on MODE_CC Condition Codes

2014-11-13 Thread Eric Botcazou
The architecture used as an example for the MODE_CC Condition Codes in the doc is the SPARC, but it turns out the quoted macros are either wrong or outdated: e.g. SELECT_CC_MODE mentions CCFPEmode and CCFPmode, but REVERSIBLE_CC_MODE says that the only mode for FP inequality comparisons is CCFPEm

Re: [PATCH] Propagate nonfreeing_call_p using ipa-pure-const (take 2)

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 09:39:42AM +0100, Jakub Jelinek wrote: > What about the: > > > I wonder if the nonfreeing_call_p function shouldn't be moved elsewhere > > > though (suggestion where), so that gimple.c doesn't need the cgraph > > > includes. > question though (maybe it is more on Richard)?

Re: [PATCH] Improve folding of bitwise ops feeding conditionals for single bit types

2014-11-13 Thread Richard Biener
On Thu, Jun 20, 2013 at 2:52 PM, Jeff Law wrote: > On 06/20/2013 04:49 AM, Andreas Schwab wrote: >> >> Jeff Law writes: >> >>> +/* { dg-final { scan-tree-dump-times "Replaced" 8 "forwprop1"} } */ >> >> >> $ grep -c Replaced forwprop-28.c.022t.forwprop1 >> 16 >> >> ;; Function test (test, funcdef_

Re: [PATCH, libgfortran] PR 60324 Unbounded stack allocations in libgfortran

2014-11-13 Thread Janne Blomqvist
On Thu, Nov 13, 2014 at 1:31 PM, FX wrote: >> Regtested on x86_64-unknown-linux-gnu, Ok for trunk? > > OK two me, with three comments: > >>* intrinsics/chmod.c (chmod_internal): New function, move logic >>here. >>(chmod_func): Call chmod_internal. > > Not sure what’s the need / benefit

Re: [PATCH] PR debug/38757 gcc does not emit DW_LANG_C99

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 12:17 PM, Jakub Jelinek wrote: > On Thu, Nov 13, 2014 at 12:08:01PM +0100, Richard Biener wrote: >> Btw, I have nothing against changing TRANSLATION_UNIT_LANGUAGE >> to DW_LANG_* directly - we only use it for choosing the language >> of the dwarf unit. This would of course

Re: [gomp4 3/6] Initial support for OpenACC memory mapping semantics.

2014-11-13 Thread Thomas Schwinge
Hi! On Tue, 14 Jan 2014 16:10:05 +0100, I wrote: > --- gcc/gimplify.c > +++ gcc/gimplify.c > @@ -69,7 +69,13 @@ enum gimplify_omp_var_data > + /* Force a specific behavior (or else, a run-time error). */ > + GOVD_MAP_FORCE = 16384, > @@ -86,7 +92,11 @@ enum omp_region_type > + /* Default to

Re: [PATCH] DW_AT_APPLE_* DWARF extensions.

2014-11-13 Thread Andrew Burgess
* Jakub Jelinek [2014-11-13 10:55:34 +0100]: > On Thu, Nov 13, 2014 at 10:35:28AM +0100, Andrew Burgess wrote: > > Am I correct to think that the include/* files are owned by GCC, and > > so the mistake here was not propagating the change to the GCC > > repository? > > Yes. > > > 2014-11-13 Shin

[PATCH] [AARCH64, NEON] Improve vcls(q?) vcnt(q?) and vld1(q?)_dup intrinsics

2014-11-13 Thread Yangfei (Felix)
Hi, This patch converts vcls(q?) vcnt(q?) and vld1(q?)_dup intrinsics to use builtin functions instead of the previous inline assembly syntax. Regtested with aarch64-linux-gnu on QEMU. Also passed the glorious testsuite of Christophe Lyon. OK for the trunk? Index: gcc/ChangeLog

bad patch for GCCPLUGIN_VERSION... help needed!

2014-11-13 Thread Basile Starynkevitch
Hello All, In https://gcc.gnu.org/ml/gcc-patches/2014-11/msg01393.html I posted a bad patch (to generate a plugin-version.c for gcc_version outside of plugin-version.h) but I can't find out what is wrong? Could any person much more familiar with autoconf that I am take a few seconds to look at it

Re: [PATCH] OpenACC for C++ front end

2014-11-13 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 03:37:08PM -0600, James Norris wrote: > 2014-11-05 James Norris > Cesar Philippidis > Thomas Schwinge > Ilmir Usmanov ... Please check formatting. I see various spots with 8 spaces instead of tabs, e.g. on + check_no_duplicate_claus

Re: [PATCH] PR debug/38757 gcc does not emit DW_LANG_C99

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 01:08:44PM +0100, Richard Biener wrote: > On Thu, Nov 13, 2014 at 12:17 PM, Jakub Jelinek wrote: > > On Thu, Nov 13, 2014 at 12:08:01PM +0100, Richard Biener wrote: > >> Btw, I have nothing against changing TRANSLATION_UNIT_LANGUAGE > >> to DW_LANG_* directly - we only use

Re: [gomp4 3/6] Initial support for OpenACC memory mapping semantics.

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 01:19:55PM +0100, Thomas Schwinge wrote: > --- include/gomp-constants.h > +++ include/gomp-constants.h > @@ -28,6 +28,9 @@ > /* Enumerated variable mapping types used to communicate between GCC and > libgomp. These values are used for both OpenMP and OpenACC. */ > >

Re: [PATCH] DW_AT_APPLE_* DWARF extensions.

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 01:21:21PM +0100, Andrew Burgess wrote: > I had a look around and couldn't find anything helpful. The best I > can offer would be the current path within the llvm source code where > these are defined. Would that be sufficient? That is not useful. The point is not to sug

Re: [PATCH, i686] Fix for asan test failures with -m32 happened after EBX enabling in PIC mode

2014-11-13 Thread H.J. Lu
On Wed, Nov 5, 2014 at 7:17 AM, Zamyatin, Igor wrote: >> > Hi! >> > >> > Following patch (moving initialization of pic_offset_table_rtx >> earlier) fixes failures for asan tests on 32 bits in PIC mode mentioned here >> - >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63534#c48 >> > >> > >> > Boo

Re: [PATCH] PR debug/38757 gcc does not emit DW_LANG_C99

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 2:07 PM, Jakub Jelinek wrote: > On Thu, Nov 13, 2014 at 01:08:44PM +0100, Richard Biener wrote: >> On Thu, Nov 13, 2014 at 12:17 PM, Jakub Jelinek wrote: >> > On Thu, Nov 13, 2014 at 12:08:01PM +0100, Richard Biener wrote: >> >> Btw, I have nothing against changing TRANSLA

Re: [gomp4] [2/3] OpenACC 2.0 support for libgomp - new tests

2014-11-13 Thread Thomas Schwinge
Hi! On Tue, 14 Oct 2014 17:11:42 +0100, Julian Brown wrote: > --- /dev/null > +++ b/libgomp/testsuite/libgomp.oacc-c/context-2.c > +float *h_X, [...] > +h_X = (float *) malloc (N * sizeof (float)); > +d_X = (float *) acc_copyin (&h_X[0], N * sizeof (float)); > +#pragma acc parall

[PATCH] Fix placement of break stmt in genmatch generated files

2014-11-13 Thread Richard Biener
This fixes an error I introduced by 2014-11-09 Richard Biener * match.pd: Add patterns convering two conversions in a row from fold-const.c. * fold-const.c (fold_unary_loc): Remove them here. * tree-ssa-forwprop.c (combine_conversions): Likewise. * genm

Re: [GRAPHITE, PATCH] Loop unroll and jam optimization

2014-11-13 Thread Mircea Namolaru
A small modification of the patch + extended comment for the mapping used for computing the separating class. The code didn't contained an adjustment of the stride with 1 (but the comment yes), that was temporary removed and forget to insert it back. The code generated now is: ISL AST generated b

gomp-constants.h (was: [gomp4 3/6] Initial support for OpenACC memory mapping semantics.)

2014-11-13 Thread Thomas Schwinge
Hi Jakub! On Thu, 13 Nov 2014 14:10:10 +0100, Jakub Jelinek wrote: > On Thu, Nov 13, 2014 at 01:19:55PM +0100, Thomas Schwinge wrote: > > --- include/gomp-constants.h > > +++ include/gomp-constants.h > > @@ -28,6 +28,9 @@ > > /* Enumerated variable mapping types used to communicate between GCC a

Re: [PATCH 0/4][Vectorizer] Reductions: replace VEC_RSHIFT_EXPR with VEC_PERM_EXPR

2014-11-13 Thread Richard Biener
On Thu, 13 Nov 2014, Alan Lawrence wrote: > Hmmm, I see. Yes, that would be another way. It has me wondering however > exactly where to draw the line, as to what permutations > TARGET_VECTORIZE_VEC_PERM_CONST_OK should allow / how much effort it should > go to. For example, should said hook return

Re: Concepts code review

2014-11-13 Thread Jason Merrill
On 11/12/2014 06:11 PM, Andrew Sutton wrote: Agreed. I'll probably start looking at this on Friday morning. Note that end of stage 1 is Saturday, as I just realized today. So the sooner the better. :) Ouch. Good thing my merge with trunk broke in unexpected ways this morning -- minimally,

Re: [ARM] Fix CLZ_DEFINED_VALUE_AT_ZERO for vector modes

2014-11-13 Thread Christophe Lyon
On 12 November 2014 14:46, Ramana Radhakrishnan wrote: > > > On 12/11/14 13:06, Christophe Lyon wrote: >> >> On 12 November 2014 04:50, Yangfei (Felix) wrote: On Wed, Oct 22, 2014 at 10:49 PM, Michael Collison wrote: > > > Patch that removes extraneous comment att

Re: gomp-constants.h (was: [gomp4 3/6] Initial support for OpenACC memory mapping semantics.)

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 02:38:06PM +0100, Thomas Schwinge wrote: > Hi Jakub! > > On Thu, 13 Nov 2014 14:10:10 +0100, Jakub Jelinek wrote: > > On Thu, Nov 13, 2014 at 01:19:55PM +0100, Thomas Schwinge wrote: > > > --- include/gomp-constants.h > > > +++ include/gomp-constants.h > > > @@ -28,6 +28,9

[PATCH][20/n] Merge from match-and-simplify, comparison "forwarding"

2014-11-13 Thread Richard Biener
This implements what forward_propagate_comparison does (and fold for a part). This leaves the first loop backwards over stmts in a BB in forwprop doing only address forward propagation (just in case we want to separate that from the "folding" transforms). It also makes the two conflicting transf

Re: [PATCH][sched-deps] Remove needless check for modified_in_p when trying to fuse two non-conditional jump insns

2014-11-13 Thread Kyrill Tkachov
On 12/11/14 23:18, Jeff Law wrote: On 11/12/14 09:23, Kyrill Tkachov wrote: Hi all, I noticed that the check for modified_in_p in sched-deps is not needed and needlessly restricts the insn pairs that we can check for macro fusion in the backend hooks. This patch removes the check. Currently th

[match-and-simplify] Merge from trunk

2014-11-13 Thread Richard Biener
2014-11-13 Richard Biener Merge from trunk r217417 through r217496. Brings back next merge piece.

Re: [PATCH] Fix PR/63841: empty constructor doesn't zero-initialize

2014-11-13 Thread Teresa Johnson
Here is the new patch. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Teresa 2014-11-13 gcc: PR tree-optimization/63841 * tree.c (initializer_zerop): A constructor with no elements does not zero initialize. gcc/testsuite: PR tree-o

Re: New patch: [AArch64] [BE] [1/2] Make large opaque integer modes endianness-safe.

2014-11-13 Thread Christophe Lyon
On 13 November 2014 11:09, David Sherwood wrote: > Hi All, > > I have successfully rebased this and tested in conjunction with a patch from > Alan Hayward ([AArch64] [BE] Fix vector load/stores to not use ld1/st1), who > should be submitting a new version shortly. Built and tested on: > > aarch64-

[PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread mliska
gcc/ChangeLog: 2014-11-12 Martin Liska * Makefile.in: New object file is added. * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID is filled up. * cgraph_summary.c: New file. * cgraph_summary.h: New file. * gengtype.c (open_base_files

[PATCH 1/4] cgraph_summary data structure introduction.

2014-11-13 Thread mliska
Following patchset introduces cgraph_summary template class that should replace custom implementation of cgraph related summaries. Idea behind the patch is to provide a generic interface one can use to register custom summary related to a cgraph_node. As you know, symbol_table provides hooks for c

[PATCH 4/4] Data structure is used for inline_summary struct.

2014-11-13 Thread mliska
gcc/ChangeLog: 2014-11-12 Martin Liska * cgraphunit.c (symbol_table::process_new_functions): inline_summary_vec is replaced with inline_summary_summary. * ipa-cp.c (ipcp_cloning_candidate_p): Usage of get_inline_summary. (devirtualization_time_bonus): Likewise.

[PATCH 3/4] First usage of cgraph_summary in ipa-prop pass.

2014-11-13 Thread mliska
gcc/ChangeLog: 2014-11-12 Martin Liska * auto-profile.c: Include cgraph_summary.h. * cgraph.c: Likewise. * cgraphbuild.c: Likewise. * cgraphclones.c: Likewise. * cgraphunit.c: Likewise. * ipa-cp.c: Likewise. * ipa-devirt.c: Likewise.

Re: [gimple-classes, committed 4/6] tree-ssa-tail-merge.c: Use gassign

2014-11-13 Thread Andrew MacLeod
On 11/13/2014 05:45 AM, Richard Biener wrote: On Thu, Nov 13, 2014 at 2:41 AM, David Malcolm wrote: On Tue, 2014-11-11 at 11:43 +0100, Richard Biener wrote: On Tue, Nov 11, 2014 at 8:26 AM, Jakub Jelinek wrote: On Mon, Nov 10, 2014 at 05:27:50PM -0500, David Malcolm wrote: On Sat, 2014-11-0

[match-and-simplify] Shuffle patterns

2014-11-13 Thread Richard Biener
Committed. Richard. 2014-11-13 Richard Biener * match.pd: Remove redundant pattern. Inline remaining stuff from match-rotate.pd and match-conversions.pd. * match-rotate.pd: Remove. * match-conversions.pd: Likewise. Index: gcc/match.pd ===

Re: [PATCH] Fix PR/63841: empty constructor doesn't zero-initialize

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 3:20 PM, Teresa Johnson wrote: > Here is the new patch. Bootstrapped and tested on > x86_64-unknown-linux-gnu. OK for trunk? Ok for trunk and branches. Thanks, Richard. > Thanks, > Teresa > > 2014-11-13 > > gcc: > PR tree-optimization/63841 > * tree.c

RE: New patch: [AArch64] [BE] [1/2] Make large opaque integer modes endianness-safe.

2014-11-13 Thread David Sherwood
Hi, I think that's because Alan Hayward's original patch had a bug in it, which he has fixed and should be submitting to gcc patches fairly soon. So it's probably best waiting until he gets a new patch up I think. Regards, David. -Original Message- From: Christophe Lyon [mailto:christoph

Re: [PATCH] Fix PR/63841: empty constructor doesn't zero-initialize

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 3:32 PM, Richard Biener wrote: > On Thu, Nov 13, 2014 at 3:20 PM, Teresa Johnson wrote: >> Here is the new patch. Bootstrapped and tested on >> x86_64-unknown-linux-gnu. OK for trunk? > > Ok for trunk and branches. Err - please fix the changelog entry wording to "A clobbe

Re: [PATCH 1/4] cgraph_summary data structure introduction.

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 3:10 PM, mliska wrote: > Following patchset introduces cgraph_summary template class that > should replace custom implementation of cgraph related summaries. > > Idea behind the patch is to provide a generic interface one can use > to register custom summary related to a cg

Re: [PATCH] Fix PR/63841: empty constructor doesn't zero-initialize

2014-11-13 Thread Teresa Johnson
On Thu, Nov 13, 2014 at 6:32 AM, Richard Biener wrote: > On Thu, Nov 13, 2014 at 3:32 PM, Richard Biener > wrote: >> On Thu, Nov 13, 2014 at 3:20 PM, Teresa Johnson wrote: >>> Here is the new patch. Bootstrapped and tested on >>> x86_64-unknown-linux-gnu. OK for trunk? >> >> Ok for trunk and bra

Re: [gimple-classes, committed 4/6] tree-ssa-tail-merge.c: Use gassign

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 3:24 PM, Andrew MacLeod wrote: > On 11/13/2014 05:45 AM, Richard Biener wrote: >> >> On Thu, Nov 13, 2014 at 2:41 AM, David Malcolm >> wrote: >>> >>> On Tue, 2014-11-11 at 11:43 +0100, Richard Biener wrote: On Tue, Nov 11, 2014 at 8:26 AM, Jakub Jelinek wrote: >

Re: [PATCH] Fix PR/63841: empty constructor doesn't zero-initialize

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 06:20:16AM -0800, Teresa Johnson wrote: > Here is the new patch. Bootstrapped and tested on > x86_64-unknown-linux-gnu. OK for trunk? > > Thanks, > Teresa > > 2014-11-13 > > gcc: > PR tree-optimization/63841 > * tree.c (initializer_zerop): A constructo

[PATCH] Disable -fuse-caller-save when -pg is active

2014-11-13 Thread Radovan Obradovic
A problem is detected with building Linux kernel on MIPS platform when both -fuse-caller-save and -pg options are present. The reason for this is that -fuse-caller-save relies on the analysis of RTL code, but when profiling is active (with -pg option) the code is instrumented by adding a call to m

Re: libsanitizer merge from upstream r221802

2014-11-13 Thread Jack Howarth
Konstantin, Applying the libsanitizer-221802.patch merge to r217456 with the proposed patch at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63534#c50, produces the following new regressions on x86_64-apple-darwin14 for asan.exp at -m32/-m64... FAIL: c-c++-common/asan/global-overflow-1.c -

Re: [PATCH 1/4] cgraph_summary data structure introduction.

2014-11-13 Thread Martin Liška
On 11/13/2014 03:33 PM, Richard Biener wrote: On Thu, Nov 13, 2014 at 3:10 PM, mliska wrote: Following patchset introduces cgraph_summary template class that should replace custom implementation of cgraph related summaries. Idea behind the patch is to provide a generic interface one can use to

Re: [PATCH][ARM] Fix names of some rounding intrinsics, impement vrndx_f32 and vrndxq_f32

2014-11-13 Thread Kyrill Tkachov
Ping. Kyrill On 04/11/14 10:56, Kyrill Tkachov wrote: Phew, This one slipped through the cracks. Ping? https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01981.html Thanks, Kyrill On 23/09/14 16:25, Kyrill Tkachov wrote: On 23/09/14 16:07, Kyrill Tkachov wrote: Hi all, Some intrinsics had the

Re: [PATCH] Fix PR/63841: empty constructor doesn't zero-initialize

2014-11-13 Thread Teresa Johnson
On Thu, Nov 13, 2014 at 6:36 AM, Jakub Jelinek wrote: > On Thu, Nov 13, 2014 at 06:20:16AM -0800, Teresa Johnson wrote: >> Here is the new patch. Bootstrapped and tested on >> x86_64-unknown-linux-gnu. OK for trunk? >> >> Thanks, >> Teresa >> >> 2014-11-13 >> >> gcc: >> PR tree-optimiz

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread Markus Trippelsdorf
On 2014.11.13 at 15:11 +0100, mliska wrote: Just two remarks: > +template > +class GTY((user)) cgraph_summary > +{ > +public: > + /* Default construction takes SYMTAB as an argument. */ > + cgraph_summary (symbol_table *symtab, bool ggc = false): m_ggc (ggc), > +m_insertion_enabled (true

Re: [PATCH] Fix PR/63841: empty constructor doesn't zero-initialize

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 3:46 PM, Teresa Johnson wrote: > On Thu, Nov 13, 2014 at 6:36 AM, Jakub Jelinek wrote: >> On Thu, Nov 13, 2014 at 06:20:16AM -0800, Teresa Johnson wrote: >>> Here is the new patch. Bootstrapped and tested on >>> x86_64-unknown-linux-gnu. OK for trunk? >>> >>> Thanks, >>> T

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-11-13 Thread Martin Liška
On 11/13/2014 03:48 PM, Markus Trippelsdorf wrote: On 2014.11.13 at 15:11 +0100, mliska wrote: Just two remarks: +template +class GTY((user)) cgraph_summary +{ +public: + /* Default construction takes SYMTAB as an argument. */ + cgraph_summary (symbol_table *symtab, bool ggc = false): m_g

Re: [PATCH] OpenACC for C front end

2014-11-13 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 03:39:44PM -0600, James Norris wrote: > * c-typeck.c (c_finish_oacc_parallel, c_finish_oacc_kernels, > c_finish_oacc_data): New functions. > (handle_omp_array_sections, c_finish_omp_clauses): Handle should be on the above line, no need to wrap too early.

Re: [PATCH] Fix PR/63841: empty constructor doesn't zero-initialize

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 06:46:23AM -0800, Teresa Johnson wrote: > >> --- tree.c (revision 217190) > >> +++ tree.c (working copy) > >> @@ -10330,6 +10330,8 @@ initializer_zerop (const_tree init) > >>{ > >> unsigned HOST_WIDE_INT idx; > >> > >> +if (TREE_CLOBBER_P (i

[PATCH][match-and-simplify] Add more binary patterns exercised by fold_stmt

2014-11-13 Thread Richard Biener
The patch also makes us allow to iterate over things in the predicate position like (for op (INTEGER_CST REAL_CST) (simplify (plus @0 op) ... Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to the branch. Richard. 2014-11-13 Richard Biener * match.pd: Implement mor

Re: [gimple-classes, committed 4/6] tree-ssa-tail-merge.c: Use gassign

2014-11-13 Thread Andrew MacLeod
On 11/13/2014 09:34 AM, Richard Biener wrote: On Thu, Nov 13, 2014 at 3:24 PM, Andrew MacLeod wrote: On 11/13/2014 05:45 AM, Richard Biener wrote: On Thu, Nov 13, 2014 at 2:41 AM, David Malcolm wrote: On Tue, 2014-11-11 at 11:43 +0100, Richard Biener wrote: On Tue, Nov 11, 2014 at 8:26 AM,

  1   2   3   >