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 cesar_philippi...@mentor.com wrote: On 07/07/2014 02:55 AM, Thomas

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 ce...@codesourcery.com 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

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 jos...@codesourcery.com: 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

[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 me

[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 rguent...@suse.de PR middle-end/61559 * match.pd:

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-lives.c (struct

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 least

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) But if you

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 ni...@redhat.com * config/rl78/divmodhi.S: Add support for the G10

[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

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: I would be

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 the need for

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,++}

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 jul...@codesourcery.com 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

[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

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 jul...@codesourcery.com Date: Mon, 22 Sep 2014 02:45:29 -0700 Subject: [PATCH 3/5] OpenACC documentation. -xx-xx Thomas Schwinge

[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 kyrylo.tkac...@arm.com *

[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 kyrylo.tkac...@arm.com

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 shinichiro.ham...@gmail.com * dwarf2.h

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 needs a

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,

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 little

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 setting +

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't

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 to be

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 dmalc...@redhat.com 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

[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

[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 patch with the redundant removed. Is

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 dmalc...@redhat.com wrote: On Tue, 2014-11-11 at 11:43 +0100, Richard Biener wrote: On Tue, Nov 11, 2014 at 8:26 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Nov 10, 2014 at 05:27:50PM -0500, David Malcolm wrote: On Sat, 2014-11-08 at 14:56

[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 paolo.carl...@oracle.com * constexpr.c (maybe_constant_folded_value): Add. * cp-tree.h (maybe_constant_folded_value): Declare it. *

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

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 v...@linux.vnet.ibm.com 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 v...@linux.vnet.ibm.com wrote: On Tue, Nov

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) { foo (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 ja...@redhat.com 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

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 ja...@redhat.com 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

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 hongjiu...@intel.com 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

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 alan.lawre...@arm.com 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

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

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 alan.lawre...@arm.com 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

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 alan.lawre...@arm.com 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

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 alan.lawre...@arm.com 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

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

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)? Tried

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 l...@redhat.com wrote: On 06/20/2013 04:49 AM, Andreas Schwab wrote: Jeff Law l...@redhat.com writes: +/* { dg-final { scan-tree-dump-times Replaced 8 forwprop1} } */ $ grep -c Replaced forwprop-28.c.022t.forwprop1 16 ;; Function test (test,

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 fxcoud...@gmail.com 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 /

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 ja...@redhat.com 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

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 ja...@redhat.com [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

[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:

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

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 jnor...@codesourcery.com Cesar Philippidis ce...@codesourcery.com Thomas Schwinge tho...@codesourcery.com Ilmir Usmanov i.usma...@samsung.com ... Please check formatting.

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 ja...@redhat.com 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

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

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 igor.zamya...@intel.com 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

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 ja...@redhat.com 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 ja...@redhat.com wrote: On Thu, Nov 13, 2014 at 12:08:01PM +0100, Richard Biener wrote: Btw, I have nothing

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 jul...@codesourcery.com 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));

[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 rguent...@suse.de * 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):

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

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 ja...@redhat.com 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

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 ramana.radhakrish...@arm.com wrote: On 12/11/14 13:06, Christophe Lyon wrote: On 12 November 2014 04:50, Yangfei (Felix) felix.y...@huawei.com wrote: On Wed, Oct 22, 2014 at 10:49 PM, Michael Collison michael.colli...@linaro.org wrote:

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 ja...@redhat.com wrote: On Thu, Nov 13, 2014 at 01:19:55PM +0100, Thomas Schwinge wrote: --- include/gomp-constants.h +++ include/gomp-constants.h @@ -28,6

[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

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

[match-and-simplify] Merge from trunk

2014-11-13 Thread Richard Biener
2014-11-13 Richard Biener rguent...@suse.de 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-13tejohn...@google.com gcc: PR tree-optimization/63841 * tree.c (initializer_zerop): A constructor with no elements does not zero initialize.

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 david.sherw...@arm.com 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

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

2014-11-13 Thread mliska
gcc/ChangeLog: 2014-11-12 Martin Liska mli...@suse.cz * 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

[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

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

2014-11-13 Thread mliska
gcc/ChangeLog: 2014-11-12 Martin Liska mli...@suse.cz * 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.

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

2014-11-13 Thread mliska
gcc/ChangeLog: 2014-11-12 Martin Liska mli...@suse.cz * 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:

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 dmalc...@redhat.com wrote: On Tue, 2014-11-11 at 11:43 +0100, Richard Biener wrote: On Tue, Nov 11, 2014 at 8:26 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Nov 10, 2014 at 05:27:50PM -0500,

[match-and-simplify] Shuffle patterns

2014-11-13 Thread Richard Biener
Committed. Richard. 2014-11-13 Richard Biener rguent...@suse.de * 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 tejohn...@google.com 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-13tejohn...@google.com gcc: PR

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

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 richard.guent...@gmail.com wrote: On Thu, Nov 13, 2014 at 3:20 PM, Teresa Johnson tejohn...@google.com 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

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 mli...@suse.cz 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

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 richard.guent...@gmail.com wrote: On Thu, Nov 13, 2014 at 3:32 PM, Richard Biener richard.guent...@gmail.com wrote: On Thu, Nov 13, 2014 at 3:20 PM, Teresa Johnson tejohn...@google.com wrote: Here is the new patch. Bootstrapped and tested on

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 amacl...@redhat.com wrote: On 11/13/2014 05:45 AM, Richard Biener wrote: On Thu, Nov 13, 2014 at 2:41 AM, David Malcolm dmalc...@redhat.com wrote: On Tue, 2014-11-11 at 11:43 +0100, Richard Biener wrote: On Tue, Nov 11, 2014 at 8:26 AM, Jakub

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-13tejohn...@google.com gcc: PR tree-optimization/63841 * tree.c (initializer_zerop): A

[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

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 mli...@suse.cz 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

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 ja...@redhat.com 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-13tejohn...@google.com gcc:

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 T +class GTY((user)) cgraph_summary T * +{ +public: + /* Default construction takes SYMTAB as an argument. */ + cgraph_summary (symbol_table *symtab, bool ggc = false): m_ggc (ggc), +m_insertion_enabled

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 tejohn...@google.com wrote: On Thu, Nov 13, 2014 at 6:36 AM, Jakub Jelinek ja...@redhat.com 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

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 T +class GTY((user)) cgraph_summary T * +{ +public: + /* Default construction takes SYMTAB as an argument. */ + cgraph_summary (symbol_table *symtab, bool ggc =

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 (init)) +

[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 rguent...@suse.de *

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 amacl...@redhat.com wrote: On 11/13/2014 05:45 AM, Richard Biener wrote: On Thu, Nov 13, 2014 at 2:41 AM, David Malcolm dmalc...@redhat.com wrote: On Tue, 2014-11-11 at 11:43 +0100, Richard Biener

  1   2   3   >