[AArch64] Fix Prefetch ICE

Hi, Please find attached the patch that fixes an ICE for prefetch. The predicate is too lose for the constraints. Hence, the patch tightens up the predicate to be exactly as constraint allows, avoids a “reload” and allows better code generation.   Submitted on behalf of Andrew Pinski. Thanks, N

Re: [PATCH] Don't create superfluous parm in expand_omp_taskreg

On 24/09/15 08:23, Thomas Schwinge wrote: Hi Tom! On Tue, 11 Aug 2015 20:53:39 +0200, Tom de Vries wrote: Don't create superfluous parm in expand_omp_taskreg 2015-08-11 Tom de Vries * omp-low.c (expand_omp_taskreg): If in ssa, set rhs of parcopy stmt to parm_decl, rather t

Re: CSE pass prevents loop-invariant motion

On 09/15/2015 05:25 PM, Jeff Law wrote: > On 09/15/2015 06:11 AM, Robin Dapp wrote: >> Hi, >> >> recently, I came across a problem that keeps a load instruction in a >> loop although it is loop-invariant. [..] > You might want to check your costing model -- cprop is supposed to look > at costs to h

Re: [PATCH] Don't create superfluous parm in expand_omp_taskreg

Hi Tom! On Tue, 11 Aug 2015 20:53:39 +0200, Tom de Vries wrote: > [ was: Re: [committed, gomp4] Fix release_dangling_ssa_names ] > > On 05/08/15 13:13, Richard Biener wrote: > > On Wed, 5 Aug 2015, Tom de Vries wrote: > > > >> On 05/08/15 11:30, Richard Biener wrote: > >>> On Wed, 5 Aug 2015, To

Re: using scratchpads to enhance RTL-level if-conversion: the new patch is almost ready to be prepared for merging to trunk, but not 100% ready yet

On 09/23/2015 06:11 PM, Abe wrote: The patch can pass the bootstrap stage2-to-stage3 comparison [same platform] *_if_* I prevent the "bootstrap-debug" value for BUILD_CONFIG from being automatically chosen, e.g. via "--with-build-config=bootstrap=time" during configuration. With the default "BUIL

Re: [RS6000] Don't pass --oformat to ld

On Wed, 2015-09-02 at 11:05 +0930, Alan Modra wrote: > bugzilla.redhat.com/show_bug_cgi?id=1255946 shows that gcc built with > both powerpc64-linux and powerpc64le-linux support passes wrong linker > options when trying to link in the non-default endian. A --oformat > option coming from LINK_TARGE

Re: [PATCH] rs6000: Fix -mdebug=stack code for spe_gp_offset

On Wed, Sep 23, 2015 at 7:35 PM, Segher Boessenkool wrote: > This seems like an obvious typo. I cannot test SPE, but I noticed > this offset shows up in the debug output for normal configurations. > The condition is inverted, compared to all similar ones. > > Is this okay for trunk? > > > Segher

[PATCH] DWARF support for AIX v4

Richard and Richard, Appended is the updated version of the DWARF support patch for AIX. I still can split out the length computation into a separate helper function, but, as I mentioned, it won't apply to the instance that uses a delta of two labels. This version sets have_macinfo to False and

[PATCH] update a few places for the change from gimple_statement_base to gimple

From: Trevor Saunders Hi, This fixes up a few remaining references to gimple_statement_base that were just brought up. bootstrapped on x86_64-linux-gnu, but the only non comment / doc change is gdbhooks.py, ok? Trev gcc/ChangeLog: 2015-09-23 Trevor Saunders * doc/gimple.texi: U

Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)

On Wed, 2015-09-23 at 15:36 +0200, Richard Biener wrote: > On Wed, Sep 23, 2015 at 3:19 PM, Michael Matz wrote: > > Hi, > > > > On Tue, 22 Sep 2015, David Malcolm wrote: > > > >> The drawback is that it could bloat the ad-hoc table. Can the ad-hoc > >> table ever get smaller, or does it only ever

[committed, pa] Add long long support to config/pa/linux-atomic.c

The attached change re-integrates long long support into linux-atomic.c. This is possible due a kernel fix to the LWS syscall and a middle-end fix to expand_atomic_compare_and_swap. The patch corrects the inverted value returned by __sync_bool_compare_and_swap_##WIDTH. It revises the return va

[PATCH] rs6000: Fix -mdebug=stack code for spe_gp_offset

This seems like an obvious typo. I cannot test SPE, but I noticed this offset shows up in the debug output for normal configurations. The condition is inverted, compared to all similar ones. Is this okay for trunk? Segher 2015-09-23 Segher Boessenkool * config/rs6000/rs6000.c (deb

Re: [RFC PATCH] parse #pragma GCC diagnostic in libcpp

On Sun, 20 Sep 2015, Manuel López-Ibáñez wrote: > PING^2: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02414.html > > On 21 August 2015 at 19:41, Manuel López-Ibáñez wrote: > > Any comments on this? > > https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02414.html > > > > I don't see any other way

[PATCH 4/4] bb-reorder: Documentation updates

This updates the documentation for the new option and new defaults. 2015-09-23 Segher Boessenkool * doc/invoke.texi (Optimization Options): Add -freorder-blocks-algorithm=. (Optimize Options) <-O>: Add -freorder-blocks. <-O2>: Remove -freorder-blocks. Add -fr

[PATCH 3/4] bb-reorder: Add -freorder-blocks-algorithm= and wire it up

This adds an -freorder-blocks-algorithm=[simple|stc] flag, with "simple" as default. For -O2 and up (except -Os) it is switched to "stc" instead. Targets that never want STC can override this. This changes -freorder-blocks to be on at -O1 and up (was -O2 and up). In effect, the changes are for -

[PATCH 2/4] bb-reorder: Add the "simple" algorithm

This is the meat of this series: a new algorithm to do basic block reordering. It uses the simple greedy approach to maximum weighted matching, where the weights are the predicted execution frequency of the edges. This always finds a solution that is within a factor two of optimal, if you disrega

[PATCH 1/4] bb-reorder: Split out STC

This first patch simply factors code a little bit. 2015-09-23 Segher Boessenkool * bb-reorder.c (reorder_basic_blocks_software_trace_cache): New function, factored out from ... (reorder_basic_blocks): ... here. --- gcc/bb-reorder.c | 29 ++---

[PATCH 0/4] bb-reorder: Add the "simple" algorithm

The current basic block reordering always uses the "software trace cache" algorithm. That has a few problems: 1) It increases code size substantially; this makes it not suitable for -O1 or -Os, and not at all for some architectures; 2) but it is enabled for -Os and all targets; 3) and -O1 gets no

Re: [PATCH, ARM]: Fix static interworking call

On 21 September 2015 at 16:15, Christian Bruel wrote: > > > On 09/18/2015 05:03 PM, Richard Earnshaw wrote: > >>> Index: attr_thumb-static2.c >>> === >>> --- attr_thumb-static2.c(revision 227904) >>> +++ attr_thumb-static2.c

libgo patch committed: Use =, not ==, in mksysinfo.sh

As suggested in http://gcc.gnu.org/PR67695 , we should not use test with == in mksysinfo.sh. This patch fixes it. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE

Re: [gomp4 0/8] NVPTX: initial OpenMP offloading

On Wed, 23 Sep 2015, Bernd Schmidt wrote: > I have two major concerns here. Can I ask you how much experience you have > with GPU programming and ptx? I'd say I have a good understanding of the programming model and nvidia hardware architecture, having used CUDA tools and payed attention to r&d ne

Re: [PR64164] drop copyrename, integrate into expand

On Sep 18, 2015, Alan Lawrence wrote: > With the latest git commit 2b27ef197ece54c4573c5a748b0d40076e35412c on > branch aoliva/pr64164, I am now able to build a cross toolchain for > aarch64 and aarch64_be, and can confirm the ABI failure is fixed on > the branch. Thanks for the confirmation. I

Re: Elimitate duplication of get_catalogs in different abi

On 05/09/2015 23:02, François Dumont wrote: > On 22/08/2015 14:24, Daniel Krügler wrote: >> 2015-08-21 23:11 GMT+02:00 François Dumont : >>> I think I found a better way to handle this problem. It is c++locale.cc >>> that needs to be built with --fimplicit-templates. I even think that the >>> *_cow

Re: New post-LTO OpenACC pass

On 09/23/15 14:51, Bernd Schmidt wrote: On 09/23/2015 08:42 PM, Nathan Sidwell wrote: As I feared, builtin folding occurs in several places. In particular its first call is very early on in the host compiler, which is far too soon. We have to defer folding until we know whether we're doing ho

Re: [ubsan PATCH] Fix uninitialized var issue (PR sanitizer/64906)

On 09/23/2015 06:07 PM, Marek Polacek wrote: Given that the code above seems to be useless now, I think let's put this patch in as-is, backport it to gcc-5, then remove those redundant hunks on trunk and add the testcase above. Do you agree? Sounds reasonable. If you can find a point in the hi

Re: New post-LTO OpenACC pass

On 09/23/2015 08:42 PM, Nathan Sidwell wrote: As I feared, builtin folding occurs in several places. In particular its first call is very early on in the host compiler, which is far too soon. We have to defer folding until we know whether we're doing host or device compilation. Doesn't somet

Re: [gomp4] remap variables inside gang, worker, vector and collapse clauses

On 09/23/2015 11:26 AM, Thomas Schwinge wrote: > On Wed, 23 Sep 2015 10:57:40 -0700, Cesar Philippidis > wrote: >> On 09/23/2015 10:42 AM, Cesar Philippidis wrote: >> | Gang, worker, vector and collapse all contain optional arguments which >> | may be used during loop expansion. In OpenACC, those

Re: [RFC] Try vector as a new representation for vector masks

On 09/23/2015 06:53 AM, Richard Biener wrote: > I think independent improvements are > > 1) remove (most) of the bool patterns from the vectorizer > > 2) make VEC_COND_EXPR not have a GENERIC comparison embedded > > (same for COND_EXPR?) Careful. The reason that COND_EXPR have embedded compa

Re: New post-LTO OpenACC pass

On 09/23/15 08:58, Bernd Schmidt wrote: On 09/23/2015 02:14 PM, Nathan Sidwell wrote: On 09/23/15 06:59, Bernd Schmidt wrote: On 09/22/2015 05:16 PM, Nathan Sidwell wrote: +if (gimple_call_builtin_p (call, BUILT_IN_ACC_ON_DEVICE)) + /* acc_on_device must be evaluated at compile time f

Re: [gomp4 0/8] NVPTX: initial OpenMP offloading

On 09/23/2015 07:22 PM, Alexander Monakov wrote: This patch series implements some minimally required changes to have OpenMP offloading working for NVPTX target on the gomp4 branch. '#pragma omp target' and data updates should work, but all parallel execution functionality remains stubbed out (u

Re: [gomp4] remap variables inside gang, worker, vector and collapse clauses

Hi! On Wed, 23 Sep 2015 10:57:40 -0700, Cesar Philippidis wrote: > On 09/23/2015 10:42 AM, Cesar Philippidis wrote: > | Gang, worker, vector and collapse all contain optional arguments which > | may be used during loop expansion. In OpenACC, those expressions could > | contain variables I'm fai

Re: [C/C++ PATCH] RFC: Implement -Wduplicated-cond (PR c/64249) (take

On Wed, Sep 23, 2015 at 12:21:35PM -0600, Jeff Law wrote: > On 09/23/2015 10:32 AM, Marek Polacek wrote: > >On Tue, Sep 22, 2015 at 03:33:34PM -0600, Martin Sebor wrote: > >>It's fine by me (for whatever it's worth). > > > >Thanks. Let's wait if Jason/Joseph or anyone else wants to chime in. > > >

Re: [PATCH] PR28901 -Wunused-variable ignores unused const initialised variables

On 09/18/2015 08:29 PM, Martin Sebor wrote: I guess it is not the 'const' I think should be handled special but the 'static'. Having unused static variables (const or not) declared in a header file but unused seems reasonable since the header file may be included in multiple .c files each of whi

Re: [PATCH][tree-inline][obvious] Delete redundant count_insns_seq

On 09/23/2015 10:05 AM, Kyrill Tkachov wrote: Hi all, I notice that the functions count_insns_seq and estimate_num_insns_seq perform the exact same function for exactly the same arguments. It's redundant to keep both around. I've decided to delete count_insns_seq and replace its one use by estim

[C++ PATCH] Fix small typos in the coding rule enforcement warnings.

Tested on Linux-PPC64, committed as obvious. /cp 2015-09-23 Ville Voutilainen Fix small typos in the coding rule enforcement warnings. * parser.c (cp_parser_namespace_definition): Replace 'namepace' with 'namespace'. /testsuite 2015-09-23 Ville Voutilainen Fix small typos

Re: [C/C++ PATCH] RFC: Implement -Wduplicated-cond (PR c/64249) (take

On 09/23/2015 10:32 AM, Marek Polacek wrote: On Tue, Sep 22, 2015 at 03:33:34PM -0600, Martin Sebor wrote: It's fine by me (for whatever it's worth). Thanks. Let's wait if Jason/Joseph or anyone else wants to chime in. Btw., if you're unhappy about having to wipe out the whole chain after e

Re: [gomp4] remap variables inside gang, worker, vector and collapse clauses

On 09/23/2015 10:42 AM, Cesar Philippidis wrote: > I've applied this patch to gomp-4_0-branch. This patch, that is. Cesar 2015-09-23 Cesar Philippidis gcc/ * omp-low.c (lower_omp_for): Remap any variables present in OMP_CLAUSE_GANG, OMP_CLAUSE_WORKER, OMP_CLAUSE_VECTOR and OMP_CLAUSE_CO

[gomp4] oacc xform updates

I've committed this patch to change all the OACC hooks to take a gcall * rather than 'gimple'. mainline has changed the type of 'gimple', and we know we're passing a call anyway. Also updated the rescanning to be more straightforwards. nathan 2015-09-23 Nathan Sidwell * target.def: GOACC

Re: [PATCH] Preserve restrict dependence info in FRE/PRE

On September 23, 2015 3:00:51 PM GMT+02:00, Richard Biener wrote: >*** copy_reference_ops_from_ref (tree ref, v >*** 816,826 >--- 828,846 > temp.op0 = TREE_OPERAND (ref, 1); > if (tree_fits_shwi_p (TREE_OPERAND (ref, 1))) > temp.off = tree_to_shwi

[gomp4] remap variables inside gang, worker, vector and collapse clauses

Gang, worker, vector and collapse all contain optional arguments which may be used during loop expansion. In OpenACC, those expressions could contain variables, but those variables aren't always getting remapped automatically. This patch remaps those variables inside lower_omp_loop. Note that I di

Re: (patch,rfc) s/gimple/gimple */

Hi! On Sat, 19 Sep 2015 20:55:35 -0400, Trevor Saunders wrote: > On Fri, Sep 18, 2015 at 09:32:37AM -0600, Jeff Law wrote: > > On 09/18/2015 07:32 AM, Trevor Saunders wrote: > > >On Wed, Sep 16, 2015 at 03:11:14PM -0400, David Malcolm wrote: > > >>On Wed, 2015-09-16 at 09:16 -0400, Trevor Saunde

[gomp4 5/8] libgomp: provide sem.h, mutex.h, ptrlock.h on nvptx

This patch provides minimal non-stub implementations for libgomp mutex/ptrlock/semaphore, using atomic ops and busy waiting. The goal here is to at least provide stub struct declarations necessary to unbreak libgomp.h. Atomics with busy waiting seems to be the only way to provide such primitives

[gomp4 8/8] libgomp: provide ICVs via env.c on nvptx

This patch ports env.c to NVPTX. It drops all environment parsing routines since there's no "environment" on the device. For now, the useful effect of the patch is providing 'omp_is_initial_device' to distinguish host execution from target execution in user code. Several functions use gomp_icv,

[gomp4 6/8] libgomp: provide stub bar.h on nvptx

This stub header only provides empty struct gomp_barrier_t. For now I've punted on providing a minimally-correct implementation. * config/nvptx/bar.h: New file. --- libgomp/config/nvptx/bar.h | 38 ++ 1 file changed, 38 insertions(+) create mode 10064

[gomp4 7/8] libgomp: work around missing pthread_attr_t on nvptx

Although newlib headers define most pthreads types, pthread_attr_t is not available. Macro-replace it by 'void' to keep the prototype of gomp_init_thread_affinity unchanged, and do not declare gomp_thread_attr. * libgomp.h: Define pthread_attr_t to void on NVPTX. --- libgomp/libgomp.h |

[gomp4 2/8] nvptx mkoffload: do not restrict to OpenACC

This patch allows to meaningfully invoke mkoffload with -fopenmp. The check for -fopenacc flag is specific to gomp4 branch: trunk does not have it. * config/nvptx/mkoffload.c (main): Do not check for -fopenacc. --- gcc/config/nvptx/mkoffload.c | 7 ++- 1 file changed, 2 insertions(+)

[gomp4 0/8] NVPTX: initial OpenMP offloading

Hello, This patch series implements some minimally required changes to have OpenMP offloading working for NVPTX target on the gomp4 branch. '#pragma omp target' and data updates should work, but all parallel execution functionality remains stubbed out (uses of '#pragma omp parallel' in target reg

[gomp4 3/8] libgomp: provide target-to-host fallback diagnostic

This patch allows to see when target regions are executed on host with GOMP_DEBUG=1 in the environment. * target.c (GOMP_target): Use gomp_debug on fallback path. --- libgomp/target.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libgomp/target.c b/libgomp/target.c index 6ca80ad..1c

[gomp4 4/8] libgomp: minimal OpenMP support in plugin-nvptx.c

This is a minimal patch for NVPTX OpenMP offloading, using Jakub's initial implementation. It allows to successfully run '#pragma omp target', without any parallel execution: 1 team of 1 thread is spawned on the device, and target regions with '#pragma omp parallel' will fail with a link error.

[gomp4 1/8] nvptx: remove assumption of OpenACC attrs presence

This patch makes one OpenACC-specific path in nvptx_record_offload_symbol optional. * config/nvptx/nvptx.c (nvptx_record_offload_symbol): Allow missing OpenACC attributes. --- gcc/config/nvptx/nvptx.c | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff

libgo patch committed: rewrite lfstack to look more like gc code

This patch by Michael Hudson-Doyle rewrites the lfstack code in libgo to look more like that in the gc library. It also fixes it for arm64. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE

Re: [C/C++ PATCH] RFC: Implement -Wduplicated-cond (PR c/64249) (take

On Tue, Sep 22, 2015 at 03:33:34PM -0600, Martin Sebor wrote: > It's fine by me (for whatever it's worth). Thanks. Let's wait if Jason/Joseph or anyone else wants to chime in. > Btw., if you're unhappy about having to wipe out the whole chain > after every side-effect it occurred to me that it

[patch] Reduce space and time overhead of std::thread

For PR 65393 I avoided some unnecessary shared_ptr copies while launching a std::thread. This goes further and avoids shared_ptr entirely, using unique_ptr instead. This reduces the memory overhead of a std::thread by 32 bytes (on 64-bit) and avoids any reference-count updates. The downside is it

Re: [ubsan PATCH] Fix uninitialized var issue (PR sanitizer/64906)

On Wed, Sep 23, 2015 at 01:08:53PM +0200, Bernd Schmidt wrote: > On 09/22/2015 05:11 PM, Marek Polacek wrote: > > >diff --git gcc/c-family/c-ubsan.c gcc/c-family/c-ubsan.c > >index e0cce84..d2bc264 100644 > >--- gcc/c-family/c-ubsan.c > >+++ gcc/c-family/c-ubsan.c > >@@ -104,6 +104,7 @@ ubsan_inst

[PATCH][tree-inline][obvious] Delete redundant count_insns_seq

Hi all, I notice that the functions count_insns_seq and estimate_num_insns_seq perform the exact same function for exactly the same arguments. It's redundant to keep both around. I've decided to delete count_insns_seq and replace its one use by estimate_num_insns_seq. Bootstrapped and tested o

Re: [PATCH] vectorizing conditional expressions (PR tree-optimization/65947)

On 18/09/2015 14:53, "Alan Hayward" wrote: > > >On 18/09/2015 14:26, "Alan Lawrence" wrote: > >>On 18/09/15 13:17, Richard Biener wrote: >>> >>> Ok, I see. >>> >>> That this case is already vectorized is because it implements MAX_EXPR, >>> modifying it slightly to >>> >>> int foo (int *a) >>>

Re: [gomp4] lock/unlock internal fn

Hi! On Wed, 23 Sep 2015 10:19:15 -0400, Nathan Sidwell wrote: > On 09/23/15 10:16, Thomas Schwinge wrote: > > On Wed, 23 Sep 2015 08:40:51 -0400, Nathan Sidwell > > wrote: > >> On 09/23/15 05:27, Thomas Schwinge wrote: > >>> On Mon, 17 Aug 2015 15:30:16 -0400, Nathan Sidwell wrote: > I'v

Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)

On 09/23/2015 07:47 AM, Michael Matz wrote: Hi, On Wed, 23 Sep 2015, Richard Biener wrote: The issue we have with LTO is that the linemap gets populated in quite random order and thus we repeatedly switch files (we've mitigated this somewhat for GCC 5). Yes. We also considered dropping col

Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

Hi! I clarified the -foffload usage: . On Wed, 23 Sep 2015 00:23:50 +0200, Bernd Schmidt wrote: > On 09/22/2015 02:02 PM, Thomas Schwinge wrote: > > > > gcc/ > > * gcc.c (handle_foffload_option): Don't lose

Re: [PATCH] Add new hooks ASM_OUTPUT_START_FUNCTION_HEADER ...

On Tue, Sep 22, 2015 at 01:56:15PM -0600, Jeff Law wrote: > Is there some good reason these aren't hooks? No, that was just inobservance. New version attached. Would it be preferrable to initialize the hooks with a NULL pointer and test the pointer before calling them? (That way the changes to

Re: [gomp4] lock/unlock internal fn

On 09/23/15 10:16, Thomas Schwinge wrote: Hi Nathan! On Wed, 23 Sep 2015 08:40:51 -0400, Nathan Sidwell wrote: On 09/23/15 05:27, Thomas Schwinge wrote: On Mon, 17 Aug 2015 15:30:16 -0400, Nathan Sidwell wrote: I've committed this patch to add a new pair of internal functions. These will

Re: [gomp4] lock/unlock internal fn

Hi Nathan! On Wed, 23 Sep 2015 08:40:51 -0400, Nathan Sidwell wrote: > On 09/23/15 05:27, Thomas Schwinge wrote: > > On Mon, 17 Aug 2015 15:30:16 -0400, Nathan Sidwell wrote: > >> I've committed this patch to add a new pair of internal functions. These > >> will > >> be used in implementing r

Re: Refactor omp_reduction_init: omp_reduction_init_op

gcc/ * omp-low.h (omp_reduction_init_op): Declare. * omp-low.c (omp_reduction_init_op): New, broken out of ... (omp_reduction_init): ... here. Call it. * tree-parloops.c (initialize_reductions): Use omp_reduction_init_op. That looks ok. Bernd

Refactor omp_reduction_init: omp_reduction_init_op (was: [gomp4] ptx reduction simplification)

Hi! On Tue, 22 Sep 2015 11:11:59 -0400, Nathan Sidwell wrote: > On 09/22/15 11:10, Thomas Schwinge wrote: > > On Fri, 18 Sep 2015 20:05:48 -0400, Nathan Sidwell wrote: > >> I've committed this patch to rework and simplify [...] > >> the reduction lowering hooks. > >> > >> The current implementa

[PATCH] Fix PR67662

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-09-23 Richard Biener PR middle-end/67662 * fold-const.c (fold_binary_loc): Do not reassociate two vars with undefined overflow unless they will cancel out. * gcc.dg/ubsan/pr67662.c:

Re: [RFC] Try vector as a new representation for vector masks

On Wed, Sep 23, 2015 at 3:41 PM, Ilya Enkovich wrote: > 2015-09-18 16:40 GMT+03:00 Ilya Enkovich : >> 2015-09-18 15:22 GMT+03:00 Richard Biener : >>> >>> I was thinking about targets not supporting generating vec >>> (of whatever mode) from a comparison directly but only via >>> a COND_EXPR. >> >>

Re: patch for PR61578

On Tue, Sep 01, 2015 at 03:39:19PM -0400, Vladimir Makarov wrote: > The following patch is for > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61578 > > The patch was bootstrapped and tested on x86 and x86-64. > > Committed as rev. 227382. > > 2015-09-01 Vladimir Makarov > >

Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)

Hi, On Wed, 23 Sep 2015, Richard Biener wrote: > The issue we have with LTO is that the linemap gets populated in quite > random order and thus we repeatedly switch files (we've mitigated this > somewhat for GCC 5). Yes. > We also considered dropping column info (and would drop range info) as

Re: [RFC] Try vector as a new representation for vector masks

On Fri, Sep 18, 2015 at 3:40 PM, Ilya Enkovich wrote: > 2015-09-18 15:22 GMT+03:00 Richard Biener : >> On Thu, Sep 3, 2015 at 3:57 PM, Ilya Enkovich wrote: >>> 2015-09-03 15:11 GMT+03:00 Richard Biener : On Thu, Sep 3, 2015 at 2:03 PM, Ilya Enkovich wrote: > Adding CCs. >

Re: [RFC] Masking vectorized loops with bound not aligned to VF.

On Fri, Sep 18, 2015 at 6:07 PM, Kirill Yukhin wrote: > Hello, > On 18 Sep 10:31, Richard Biener wrote: >> On Thu, 17 Sep 2015, Ilya Enkovich wrote: >> >> > 2015-09-16 15:30 GMT+03:00 Richard Biener : >> > > On Mon, 14 Sep 2015, Kirill Yukhin wrote: >> > > >> > >> Hello, >> > >> I'd like to initia

Re: [RFC] Try vector as a new representation for vector masks

2015-09-18 16:40 GMT+03:00 Ilya Enkovich : > 2015-09-18 15:22 GMT+03:00 Richard Biener : >> >> I was thinking about targets not supporting generating vec >> (of whatever mode) from a comparison directly but only via >> a COND_EXPR. > > Where may these direct comparisons come from? Vectorizer never

Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)

On Wed, Sep 23, 2015 at 3:19 PM, Michael Matz wrote: > Hi, > > On Tue, 22 Sep 2015, David Malcolm wrote: > >> The drawback is that it could bloat the ad-hoc table. Can the ad-hoc >> table ever get smaller, or does it only ever get inserted into? > > It only ever grows. > >> An idea I had is that

Re: [PATCH, i386, AVX-512] Fix iterator for k, introduce kshift[lr][bwdq].

Hello, On 22 Sep 18:14, Kirill Yukhin wrote: > Hello, > Patch in the bottom fixes iterator for k insns > since QI mode is only available for AVX-512DQ. > > It also adds support for kshift[rl][bwdq]. This patterns > will be used for mask load/store autogeneration on which > Ilya Enkovich is working

Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)

Hi, On Tue, 22 Sep 2015, David Malcolm wrote: > The drawback is that it could bloat the ad-hoc table. Can the ad-hoc > table ever get smaller, or does it only ever get inserted into? It only ever grows. > An idea I had is that we could stash short ranges directly into the 32 > bits of locatio

[PATCH] Preserve restrict dependence info in FRE/PRE

I noticed we don't handle secondary effects of restrict in FRE when looking at another testcase from PR48885: int f (int *__restrict__ &__restrict__ p, int *p2) { *p = 1; *p2 = 2; return *p; } with the previously posted patch to improve the handling for p2 we should be able to optimize the

[PATCH] Fix testcase from PR48885

I am currently testing the following patch enabling us to optimize void test (int *a, int *b, int * restrict v) { *a = *v; *b = *v; } there is a simple case we can handle without implementing ??? from visit_loadstore. Richard. 2015-09-23 Richard Biener PR tree-optim

Re: New post-LTO OpenACC pass

On 09/23/2015 02:14 PM, Nathan Sidwell wrote: On 09/23/15 06:59, Bernd Schmidt wrote: On 09/22/2015 05:16 PM, Nathan Sidwell wrote: +if (gimple_call_builtin_p (call, BUILT_IN_ACC_ON_DEVICE)) + /* acc_on_device must be evaluated at compile time for + constant arguments. */ +

[gomp4] vector reductions

I've committed this reimplementation of the vector shuffling code. In preparing a fix for the worker reductions (to use a lockless scheme), I wanted to check VIEW_CONVERT_EXPR DTRT. Use of gimplify_assign also reduces the code size. nathan 2015-09-23 Nathan Sidwell * config/nvptx/nvptx.c

Re: [gomp4] lock/unlock internal fn

On 09/23/15 05:27, Thomas Schwinge wrote: Hi Nathan! On Mon, 17 Aug 2015 15:30:16 -0400, Nathan Sidwell wrote: I've committed this patch to add a new pair of internal functions. These will be used in implementing reductions. They'll be emitted around reduction finalization, and implement the

Re: [gomp4] Another oacc reduction simplification

On 09/23/15 04:02, Thomas Schwinge wrote: Hi! On Tue, 22 Sep 2015 11:29:37 -0400, Nathan Sidwell wrote: I've committed this patch, which simplifies the generation of openacc reduction code. Aside from the progression mentioned in

Re: New post-LTO OpenACC pass

On 09/23/15 06:59, Bernd Schmidt wrote: On 09/22/2015 05:16 PM, Nathan Sidwell wrote: +if (gimple_call_builtin_p (call, BUILT_IN_ACC_ON_DEVICE)) + /* acc_on_device must be evaluated at compile time for + constant arguments. */ + { +oacc_xform_on_device (call); +

[SH][committed] Fix PR 67391

Hi, The attached patch fixes PR 67391. Some additional reg overlapping were added to the addsi3 patterns while making LRA on SH work, but not all of them seem to be good. Removing them, seems to be working just fine. Tested on sh-elf (LRA enabled) with make -k check RUNTESTFLAGS="--target_board=

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 17/09/15 09:37 -0600, Martin Sebor wrote: On 09/17/2015 05:16 AM, Jonathan Wakely wrote: On 16/09/15 17:42 -0600, Martin Sebor wrote: I see now the first exists test will detect symlink loops in the original path. But I'm not convinced there isn't a corner case that's subject to a TOCTOU rac

[v3 patch] Fix Filesystem TS directory iterators

directory_iterator and recursive_directory_iterator fail to meet this requirement in http://wg21.link/n4099#Class-directory_iterator The directory_iterator default constructor shall create an iterator equal to the end iterator value, and this shall be the only valid iterator for the end condit

Re: [PATCH c-family/49654/49655] reject invalid options in pragma diagnostic

On Tue, Sep 22, 2015 at 08:08:28PM +0200, Manuel López-Ibáñez wrote: > + else if (!(cl_options[option_index].flags & lang_mask)) > +{ > + char * ok_langs = write_langs (cl_options[option_index].flags); > + char * bad_lang = write_langs (c_common_option_lang_mask ()); Please remove t

[v3 patch] Fix filesystem::create_directories() function

This function wasn't working properly (testing is useful!) Tested x86_64-linux, powerpc64le-linux and x86_64-dragonfly4.1, committed to trunk. commit 9f9ee62dc3e3d5a1cc825298b93afedc2eaf0aeb Author: Jonathan Wakely Date: Tue Sep 22 23:43:59 2015 +0100 Fix filesystem::create_directories()

Re: [RFC] PR tree-optimization/67628: Make tree ifcombine more symmetric and interactions with dom

On Wed, 23 Sep 2015, Kyrill Tkachov wrote: > > On 23/09/15 11:10, Richard Biener wrote: > > On Wed, 23 Sep 2015, Kyrill Tkachov wrote: > > > > > On 23/09/15 10:09, Pinski, Andrew wrote: > > > > > On Sep 23, 2015, at 1:59 AM, Kyrill Tkachov > > > > > wrote: > > > > > > > > > > > > > > > > On 2

Re: [RFC] PR tree-optimization/67628: Make tree ifcombine more symmetric and interactions with dom

On 23/09/15 11:10, Richard Biener wrote: On Wed, 23 Sep 2015, Kyrill Tkachov wrote: On 23/09/15 10:09, Pinski, Andrew wrote: On Sep 23, 2015, at 1:59 AM, Kyrill Tkachov wrote: On 22/09/15 20:31, Jeff Law wrote: On 09/22/2015 07:36 AM, Kyrill Tkachov wrote: Hi all, Unfortunately, I see a

Re: [PATCH c-family/49654/49655] reject invalid options in pragma diagnostic

On 09/22/2015 08:08 PM, Manuel López-Ibáñez wrote: Use find_opt instead of linear search through options in handle_pragma_diagnostic (PR 49654) and reject non-warning options and options not valid for the current language (PR 49655). + /* option_string + 1 to skip the initial '-' */ + unsign

[PATCH] Improve restrict handling further

The following fixes int f5 (S *__restrict x, S *__restrict y) { x->p[0] = 5; y->p[0] = 0; // { dg-final { scan-tree-dump-times "return 5" 1 "optimized" { xfail *-*-* } } } return x->p[0]; } which requires building representatives for restrict qualified pointers (as opposed to references o

Re: [ubsan PATCH] Fix uninitialized var issue (PR sanitizer/64906)

On 09/22/2015 05:11 PM, Marek Polacek wrote: diff --git gcc/c-family/c-ubsan.c gcc/c-family/c-ubsan.c index e0cce84..d2bc264 100644 --- gcc/c-family/c-ubsan.c +++ gcc/c-family/c-ubsan.c @@ -104,6 +104,7 @@ ubsan_instrument_division (location_t loc, tree op0, tree op1) } } t =

Re: New post-LTO OpenACC pass

On 09/22/2015 05:16 PM, Nathan Sidwell wrote: + if (gimple_call_builtin_p (call, BUILT_IN_ACC_ON_DEVICE)) + /* acc_on_device must be evaluated at compile time for +constant arguments. */ + { + oacc_xform_on_device (call); + rescan = true; +

Re: [Patch/ccmp] Cost instruction sequences to choose better expand order

No. Please see NOTE part of the description. AArch64 doesn't cost ccmp currently. It will be fixed by a seperate patch later. The testcase is thus marked as XFAIL. I'd prefer to do things in the right order. Your patch is approved, but please commit only after you can remove the xfail from the

Re: [gomp4, wip] remove references to ganglocal shared memory inside gcc

Hi! On Fri, 18 Sep 2015 06:51:18 -0700, Cesar Philippidis wrote: > On 09/18/2015 01:39 AM, Thomas Schwinge wrote: > > > On Tue, 1 Sep 2015 18:29:55 +0200, Tom de Vries > > wrote: > >> On 27/08/15 03:37, Cesar Philippidis wrote: > >>> - ctx->ganglocal_size_host = align_and_expand (&gl_host, h

Re: [RFC, PR target/65105] Use vector instructions for scalar 64bit computations on 32bit target

On Wed, Sep 23, 2015 at 12:19 PM, Ilya Enkovich wrote: > On 14 Sep 17:50, Uros Bizjak wrote: >> >> +(define_insn_and_split "*zext_doubleword" >> + [(set (match_operand:DI 0 "register_operand" "=r") >> + (zero_extend:DI (match_operand:SWI24 1 "nonimmediate_operand" "rm")))] >> + "!TARGET_64BIT &&

Re: [RFC, PR target/65105] Use vector instructions for scalar 64bit computations on 32bit target

On 14 Sep 17:50, Uros Bizjak wrote: > > +(define_insn_and_split "*zext_doubleword" > + [(set (match_operand:DI 0 "register_operand" "=r") > + (zero_extend:DI (match_operand:SWI24 1 "nonimmediate_operand" "rm")))] > + "!TARGET_64BIT && TARGET_STV && TARGET_SSE2" > + "#" > + "&& reload_completed

Re: [RFC] PR tree-optimization/67628: Make tree ifcombine more symmetric and interactions with dom

On Wed, 23 Sep 2015, Kyrill Tkachov wrote: > > On 23/09/15 10:09, Pinski, Andrew wrote: > > > On Sep 23, 2015, at 1:59 AM, Kyrill Tkachov > > > wrote: > > > > > > > > > > On 22/09/15 20:31, Jeff Law wrote: > > > > > On 09/22/2015 07:36 AM, Kyrill Tkachov wrote: > > > > > Hi all, > > > > > Unfo

Re: [gomp4] lock/unlock internal fn

Hi Nathan! On Mon, 17 Aug 2015 15:30:16 -0400, Nathan Sidwell wrote: > I've committed this patch to add a new pair of internal functions. These > will > be used in implementing reductions. > > They'll be emitted around reduction finalization, and implement the locking > required for the gene

Re: [RFC] PR tree-optimization/67628: Make tree ifcombine more symmetric and interactions with dom

On 23/09/15 10:09, Pinski, Andrew wrote: On Sep 23, 2015, at 1:59 AM, Kyrill Tkachov wrote: On 22/09/15 20:31, Jeff Law wrote: On 09/22/2015 07:36 AM, Kyrill Tkachov wrote: Hi all, Unfortunately, I see a testsuite regression with this patch: FAIL: gcc.dg/pr66299-2.c scan-tree-dump-not optim

[HSA] Add new gate predicate

Hello. Following patch does a small refactoring of HSA tree generation pass. Martin >From a887efce8fc6aa136a2a069ea5ddda10b4e28de6 Mon Sep 17 00:00:00 2001 From: marxin Date: Tue, 22 Sep 2015 18:58:12 +0200 Subject: [PATCH] HSA: add new gate predicate gcc/ChangeLog: 2015-09-22 Martin Liska

  1   2   >