[sched] Dump dependency graph to a dot file

Hi! The attached patch adds a procedure to dump the scheduler's dependency graph into a dot file. The patch has been bootstrapped and regtested on x86_64. Please commit if it is OK for trunk. Thanks, Nikolai 2015-11-04 Nikolai Bozhenov * sched-int.h

[PATCH] PR54224 Warn for unused internal procedures -- update testcase

Is there any objection to commit the following (see comments 21 and 22)? Dominique Index: gcc/testsuite/ChangeLog === --- gcc/testsuite/ChangeLog (revision 229793) +++ gcc/testsuite/ChangeLog (working copy) @@ -1,3 +1,9 @@

Re: [sched] Dump dependency graph to a dot file

Hi Nikolai, On 05/11/15 08:29, Nikolai Bozhenov wrote: Hi! The attached patch adds a procedure to dump the scheduler's dependency graph into a dot file. The patch has been bootstrapped and regtested on x86_64. Please commit if it is OK for trunk. Thanks, Nikolai A couple of style nits. +

[ping] Change behavior of -fsched-verbose option

Please commit if it is OK for trunk. Thanks, Nikolai On 10/22/2015 06:38 PM, Nikolai Bozhenov wrote: Hi! Currently -fsched-verbose option redirects debugging dumps to stderr if there is no dump_file for the current pass. It would be fine if there were the only scheduling pass. But for example

Re: [PATCH 1/4][AArch64] Add scheduling and cost models for Exynos M1

On Wed, Nov 04, 2015 at 05:21:03PM -0600, Evandro Menezes wrote: > Please, ignore the previous patch. This is the intended patch. > > Sorry. > > -- > Evandro Menezes > > On 11/04/2015 05:18 PM, Evandro Menezes wrote: > >This patch adds extra tuning information about AArch64 targets: > > > > *

[PATCH][ARM][cleanup] Remove uses of CONST_DOUBLE_HIGH/LOW

Hi all, This cleanup patch removes handling of CONST_DOUBLE rtxes that carry large integers. These should never be passed down from the midend and the arm backend doesn't create them. The code has been there since 2007 but the arm backend was moved to TARGET_SUPPORTS_WIDE_INT in 2014, so this

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

On 11/04/2015 05:17 PM, Andreas Arnez wrote: gcc/c/ChangeLog: PR debug/67192 * c-parser.c (c_parser_while_statement): Finish the loop before parsing ahead for misleading indentation. (c_parser_for_statement): Likewise. This is OK. Does C++ have similar issues?

Re: [ping] Change behavior of -fsched-verbose option

On 11/05/2015 09:11 AM, Nikolai Bozhenov wrote: Please commit if it is OK for trunk. Hmm, do you have a copyright assignment on file? Bernd

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

On Wed, Nov 04, 2015 at 02:26:31PM +, Ramana Radhakrishnan wrote: > > > On 03/11/15 17:09, James Greenhalgh wrote: > > On Tue, Nov 03, 2015 at 04:36:45PM +, Ramana Radhakrishnan wrote: > >> Hi, > >> > >>Now that PR63304 is fixed and we have an option to address > >> any part of the

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

Hi all, attached is a rather trivial patch to prevent multiple evaluations of a function in: allocate( array(func()) ) The patch tests whether the upper bound of the array is a function and calls gfc_evaluate_now(). Bootstrapped and regtested for x86_64-linux-gnu/f21. Ok for trunk?

[PATCH, ARM] PR target/68059 libgcc should not use __write for printing fatal error

libgcc/config/arm/linux-atomic-64bit.c uses __write to print an error message if the 64bit xchg method is not available in the kernel. __write is not part of the public libc abi. Since this code is only run on linux the write syscall can be invoked directly. And __builtin_trap is a simpler way

Re: [ping] Fix PR debug/66728

On Thu, Nov 5, 2015 at 12:45 AM, Mike Stump wrote: > > On Nov 4, 2015, at 12:50 PM, Richard Sandiford > wrote: > >> Mike Stump writes: >>> Index: dwarf2out.c >>>

Re: [PATCH/RFC/RFA] Machine modes for address printing (all targets)

On 11/04/2015 01:54 PM, Julian Brown wrote: That's not all targets by any means, but may be enough to warrant a change in the interface. I propose that: * The output_address function should have a machine_mode argument added. Bare addresses (e.g. the 'a' case in final.c) should pass

[gomp4, committed, 1/9] Move expansion of kernels region back to first omp-expand

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

[gomp4, committed, 4/9] Revert "Add pass_dominator::sese_mode_p ()"

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

[gomp4, committed, 5/9] Handle oacc function in parloops

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

Re: Fix 61441

Hi, > It seems this code has moved to fold-const.c, so the patch will need > updating. I have updated the patch and sending again. > Now, is it actually correct for integer_valued_real_single_p (now) to > treat sNaN as an integer? Looking at existing uses, they all appear to be > in match.pd:

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

On 11/05/2015 12:33 PM, Andreas Arnez wrote: Thanks again for reviewing. Are you going to look at patch #2 as well? Yeah, still thinking about that one. Does C++ have similar issues? Not this particular issue, AFAIK. But I've just looked at how C++ fares with the enhanced version of

Re: [PATCH 5/6]tree-sra.c: Fix completely_scalarize for negative array indices

On 30/10/15 10:54, Eric Botcazou wrote: > On 30/10/15 10:44, Richard Biener wrote: >> >> I think you want to use wide-ints here and >> >> wide_int idx = wi::from (minidx, TYPE_PRECISION (TYPE_DOMAIN >> (...)), TYPE_SIGN (TYPE_DOMAIN (..))); >> wide_int maxidx = ... >> >> you can then

Re: [PATCH] clarify documentation of -Q --help=optimizers

On Thu, 5 Nov 2015, Joseph Myers wrote: > On Wed, 4 Nov 2015, Martin Sebor wrote: > > > Improving the compiler output is a good idea. The attached patch > > prints "[disabled by -O0]" instead of "[enabled]" when an optimization > > option is enabled by default but when optimization (i.e., -O1 or

[gomp4, committed, 9/9] Remove BUILT_IN_GOACC_KERNELS_INTERNAL

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

[gomp4, committed] expand oacc kernels region at same time as oacc parallel region

Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the need to postpone splitting off the

[gomp4.5] Tweak OpenMP 4.5 mapping rules

Hi! This patch updates the mapping of vars to what has been ratified in OpenMP 4.5, or where left unspecified, hopefully follows the discussed intent. In particular: 1) for C++ references we map what they refer to, and on target construct privatize the references themselves (but not what

[gomp4, committed, 2/9] Update gate_oacc_kernels to handle oacc function

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

[gomp4, committed, 3/9] Revert "Add skip_stmt parm to pass_dominator::get_sese ()"

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

[gomp4, committed, 7/9] Update goacc kernels Fortran testcases

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

Re: OpenACC dimension range propagation optimization

On Wed, Nov 4, 2015 at 6:29 PM, Nathan Sidwell wrote: > On 11/04/15 05:26, Richard Biener wrote: >> >> On Tue, Nov 3, 2015 at 7:11 PM, Nathan Sidwell wrote: >>> >>> Richard, > > >> this all seems a little bit fragile and relying on implementation details? >> Is

Re: [openacc] tile, independent, default, private and firstprivate support in c/++

Hi Cesar! On Tue, 3 Nov 2015 14:16:59 -0800, Cesar Philippidis wrote: > This patch does the following to the c and c++ front ends: > * updates c_oacc_split_loop_clauses to filter out the loop clauses >from combined parallel/kernels loops > gcc/c-family/ >

Re: [PATCH] Use subreg_regno instead of subreg_regno_offset

On 11/02/2015 11:29 PM, Anatoly Sokolov wrote: @@ -5522,12 +5516,7 @@ op0 = SUBREG_REG (op0); code0 = GET_CODE (op0); if (code0 == REG && REGNO (op0) < FIRST_PSEUDO_REGISTER) - op0 = gen_rtx_REG (word_mode, - (REGNO (op0) + -

Re: [PATCH][combine][RFC] Don't transform sign and zero extends inside mults

Hi Segher, On 04/11/15 23:50, Segher Boessenkool wrote: Hi Kyrill, On Mon, Nov 02, 2015 at 02:15:27PM +, Kyrill Tkachov wrote: This patch attempts to restrict combine from transforming ZERO_EXTEND and SIGN_EXTEND operations into and-bitmask and weird SUBREG expressions when they appear

Re: [ping] Change behavior of -fsched-verbose option

On 11/05/2015 12:26 PM, Bernd Schmidt wrote: On 11/05/2015 09:11 AM, Nikolai Bozhenov wrote: Please commit if it is OK for trunk. Hmm, do you have a copyright assignment on file? Yes I do have a copyright assignment for all past and future changes to GCC (RT:828836). Thanks, Nikolai

Re: [sched] Dump dependency graph to a dot file

On 11/05/2015 12:18 PM, Kyrill Tkachov wrote: Hi Nikolai, On 05/11/15 08:29, Nikolai Bozhenov wrote: Hi! The attached patch adds a procedure to dump the scheduler's dependency graph into a dot file. The patch has been bootstrapped and regtested on x86_64. Please commit if it is OK for trunk.

[PATCH] New version of libmpx with new memmove wrapper

New version of libmpx was added. There is a new function get_bd() that allows to get bounds directory. Wrapper for memmove was modified. Now it moves data and then moves corresponding bounds directly from one bounds table to another. This approach made moving unaligned pointers possible. It also

[gomp4, committed, 6/9] Update goacc kernels C testcases

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

[gomp4, committed, 8/9] Release_defs in expand_omp_atomic_fetch_op

On 05/11/15 11:16, Tom de Vries wrote: Hi, now that we have committed -foffload-alias in gomp-4_0-branch ( https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00214.html ), we no longer need the kernels region to be a part of the original function when doing alias analysis. So, we no longer have the

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

On Thu, Nov 05 2015, Bernd Schmidt wrote: > On 11/04/2015 05:17 PM, Andreas Arnez wrote: >> >> gcc/c/ChangeLog: >> >> PR debug/67192 >> * c-parser.c (c_parser_while_statement): Finish the loop before >> parsing ahead for misleading indentation. >> (c_parser_for_statement):

Re: [PATCH] Fix c-c++-common/torture/vector-compare-1.c on powerpc

On 29 Oct 15:29, Richard Biener wrote: > > I think this should unconditionally produce the COND_EXPR and > build cst_true using build_all_ones_cst (stype). > > Ok with that change. > > Thanks, > Richard. > Here is an updated patch version. Bootstrapped and regtested on

Re: [vec-cmp, patch 1/6] Add optabs for vector comparison

2015-10-27 23:52 GMT+03:00 Jeff Law : > > Sigh. I searched for the enum type, not for CODE_FOR_nothing ;( My bad. > > If it's easy to get rid of, yes. I believe we've got 3 uses of > CODE_FOR_nothing. AFAICT in none of those cases do we care about the code > other than does it

[PATCH, PR tree-optimization/68145] Fix vectype computation in vectorizable_operation

Hi, This patch fixes a way vectype is computed in vectorizable_operation. Currently op0 is always used to compute vectype. If it is a loop invariant then its type is used to get vectype which is impossible for booleans requiring a context to correctly compute vectype. This patch uses output

Re: Fix 61441

On Thu, 5 Nov 2015, Sujoy Saraswati wrote: > > Some other places in this patch have similar issues with checking > > HONOR_SNANS && REAL_VALUE_ISNAN when they should check if the particular > > value is sNaN. > > I modified the code to check for sNaN specifically. The modified patch > is below.

Re: [openacc] tile, independent, default, private and firstprivate support in c/++

On 11/05/2015 09:13 AM, Nathan Sidwell wrote: > On 11/05/15 12:01, Thomas Schwinge wrote: > >> On Thu, 5 Nov 2015 06:47:58 -0800, Cesar Philippidis >> wrote: >>> On 11/05/2015 04:14 AM, Thomas Schwinge wrote: > >>> Sorry, I must have mis-phrased it. The spec is unclear

Re: [PATCH] PR 68192 Export AIX TLS symbols

[Explicitly copying build maintainers.] Paolo and Alexandre, Could you review and help with this patch? TLS symbols in AIX display a new, different symbol type in nm output. Libtool explicitly creates a list of exported symbols for shared libraries using nm and does not recognize the new TLS

Re: [PATCH] clarify documentation of -Q --help=optimizers

On 11/05/2015 10:10 AM, Joseph Myers wrote: On Thu, 5 Nov 2015, Martin Sebor wrote: optimiziation. We seem to have consensus that even though there might be exceptions (do we know what some of them are?) the vast majority optimizations that have -fxxx options are in fact not performed at -O0.

Re: [openacc] tile, independent, default, private and firstprivate support in c/++

Hi! Nathan, question here about clause splitting for combined OpenACC "parallel loop" and "kernels loop" constructs, in particular: #pragma acc parallel loop reduction([...]) On Thu, 5 Nov 2015 06:47:58 -0800, Cesar Philippidis wrote: > On 11/05/2015 04:14 AM,

Re: [PATCH] Add configure flag for operator new (std::nothrow)

On 5 November 2015 at 20:52, Daniel Gutson wrote: > Real use cases: statistics and logging. It's a (one time) callback > reporting that something went wrong, > but not intended to fix things e.g. by attempting to free more memory. Why can't that be done by replacing operator new with a

Re: [PATCH] clarify documentation of -Q --help=optimizers

On Thu, 5 Nov 2015, Martin Sebor wrote: > optimiziation. We seem to have consensus that even though there > might be exceptions (do we know what some of them are?) the vast > majority optimizations that have -fxxx options are in fact not > performed at -O0. I think reflecting that in the output

Re: [PATCH] PR 68192 Export AIX TLS symbols

Hi, Richi You mentioned GCC 5.3 release around the time of stage 1 ending in the GCC 5 status report: https://gcc.gnu.org/ml/gcc/2015-07/msg00197.html If the timeframe is later, we have more time to address the AIX libtool problem. Thanks, David On Thu, Nov 5, 2015 at 9:13 AM, Richard Biener

[C PATCH] Fix error-recovery with variable-sized compound literals (PR c/68090)

This fixes an error-recovery problem, where we were trying to build a C_MAYBE_CONST_EXPR even though the type is invalid. I think let's just skip this for error_mark_nodes. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2015-11-05 Marek Polacek PR c/68090

Re: [PATCH 1/4][AArch64] Add scheduling and cost models for Exynos M1

James, Since other members of the "tune_params" structure were signed integers, even though negative numbers would make no sense for most either, I followed the same pattern. Regardless, here's a patch with unsigned integers as you requested: [AArch64] Add extra tuning parameters for

Re: [PATCH] clarify documentation of -Q --help=optimizers

On 11/05/2015 03:48 AM, Alexander Monakov wrote: On Thu, 5 Nov 2015, Joseph Myers wrote: On Wed, 4 Nov 2015, Martin Sebor wrote: Improving the compiler output is a good idea. The attached patch prints "[disabled by -O0]" instead of "[enabled]" when an optimization option is enabled by default

Re: [PATCH] PR 68192 Export AIX TLS symbols

On November 5, 2015 5:38:45 PM GMT+01:00, David Edelsohn wrote: >On Thu, Nov 5, 2015 at 8:34 AM, Paolo Bonzini wrote: >> >> >> On 05/11/2015 17:28, David Edelsohn wrote: >>> [Explicitly copying build maintainers.] >>> >>> Paolo and Alexandre, >>> >>> Could

Re: [PATCH] PR 68192 Export AIX TLS symbols

On Thu, Nov 5, 2015 at 8:34 AM, Paolo Bonzini wrote: > > > On 05/11/2015 17:28, David Edelsohn wrote: >> [Explicitly copying build maintainers.] >> >> Paolo and Alexandre, >> >> Could you review and help with this patch? >> >> TLS symbols in AIX display a new, different symbol

Re: [openacc] tile, independent, default, private and firstprivate support in c/++

On 11/05/15 12:01, Thomas Schwinge wrote: On Thu, 5 Nov 2015 06:47:58 -0800, Cesar Philippidis wrote: On 11/05/2015 04:14 AM, Thomas Schwinge wrote: Sorry, I must have mis-phrased it. The spec is unclear here. There are three possible ways to interpret 'acc

Re: Division Optimization in match and simplify

On November 5, 2015 2:40:30 PM GMT+01:00, Michael Matz wrote: >Hi, > >On Wed, 4 Nov 2015, Richard Biener wrote: > >> Ah, it was _left_ shift of negative values that ubsan complains >about. > >Note that this is only for the frontend definition of shifts. I don't >see >why gimple

Re: [PATCH, ARM] PR target/68059 libgcc should not use __write for printing fatal error

On 05/11/15 15:59, Richard Earnshaw wrote: On 05/11/15 12:30, Szabolcs Nagy wrote: libgcc/config/arm/linux-atomic-64bit.c uses __write to print an error message if the 64bit xchg method is not available in the kernel. __write is not part of the public libc abi. Since this code is only run on

Re: [PATCH] PR 68192 Export AIX TLS symbols

On 05/11/2015 17:28, David Edelsohn wrote: > [Explicitly copying build maintainers.] > > Paolo and Alexandre, > > Could you review and help with this patch? > > TLS symbols in AIX display a new, different symbol type in nm output. > Libtool explicitly creates a list of exported symbols for

Re: [PING 2] [PATCH] c++/67942 - diagnose placement new buffer overflow

If we use gcc_checking_assert it won't fire in release builds; let's go with that. Okay. Attached is an updated patch with that change. Martin gcc ChangeLog 2015-11-05 Martin Sebor PR c++/67942 * invoke.texi (-Wplacement-new): Document new option. *

Re: [PATCH] PR 68192 Export AIX TLS symbols

On 05/11/2015 17:38, David Edelsohn wrote: > On Thu, Nov 5, 2015 at 8:34 AM, Paolo Bonzini wrote: >> >> >> On 05/11/2015 17:28, David Edelsohn wrote: >>> [Explicitly copying build maintainers.] >>> >>> Paolo and Alexandre, >>> >>> Could you review and help with this patch? >>>

Re: [PATCH, ARM] PR target/68059 libgcc should not use __write for printing fatal error

On 05/11/15 12:30, Szabolcs Nagy wrote: > libgcc/config/arm/linux-atomic-64bit.c uses __write to print an error > message if the 64bit xchg method is not available in the kernel. > > __write is not part of the public libc abi. Since this code is only > run on linux the write syscall can be

Re: [openacc] tile, independent, default, private and firstprivate support in c/++

Hi! On Wed, Nov 04, 2015 at 09:55:49AM -0800, Cesar Philippidis wrote: So, you are going to deal with gang parsing incrementally? Fine with me. > +/* OpenACC 2.0: > + tile ( size-expr-list ) */ > + > +static tree > +c_parser_oacc_clause_tile (c_parser *parser, tree list) > +{ > + tree c,

Re: [PATCH] clarify documentation of -Q --help=optimizers

On Thu, Nov 05, 2015 at 08:58:25AM -0700, Martin Sebor wrote: > I don't think that reiterating in a condensed form what the manual > doesn't make clear in many more words will help. First, even users > who do find the relevant text in the manual often misunderstand it. > Others are misled by the

Re: [C PATCH] Fix error-recovery with variable-sized compound literals (PR c/68090)

On Thu, 5 Nov 2015, Marek Polacek wrote: > This fixes an error-recovery problem, where we were trying to build > a C_MAYBE_CONST_EXPR even though the type is invalid. I think let's > just skip this for error_mark_nodes. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? OK. -- Joseph

[gomp4] Merge trunk r229764 (2015-11-04) into gomp-4_0-branch

Hi! Committed to gomp-4_0-branch in r229821: commit bed2efe185cf13a52b9cf87ee52d74265f97d97b Merge: bb6471e bf3a27b Author: tschwinge Date: Thu Nov 5 18:02:08 2015 + svn merge -r 229562:229764 svn+ssh://gcc.gnu.org/svn/gcc/trunk

Re: [PATCH] clarify documentation of -Q --help=optimizers

On Thu, 5 Nov 2015, Martin Sebor wrote: > Yes, I agree that printing "disabled by -O0" when -fexceptions is > explicitly enabled on the command line is not right if the option > isn't actually affected by -O0. This would be prevented if we knew > which of the -fxxx options are or aren't

Re: [PATCH] clarify documentation of -Q --help=optimizers

On 11/05/2015 11:07 AM, Joseph Myers wrote: On Thu, 5 Nov 2015, Martin Sebor wrote: There are several options marked Optimization that are actually matters of language semantics that I don't think have anything to do with -O options, e.g. -fexceptions and -fcx-limited-range. The Optimization

Re: [PATCH 6/6] Make SRA replace constant-pool loads

On 3 November 2015 at 14:01, Richard Biener wrote: > > Hum. I still wonder why we need all this complication ... Well, certainly I'd love to make it simpler, and if the complication is because I've gone about trying to deal with especially Ada in the wrong way... >

Re: [PATCH] Add configure flag for operator new (std::nothrow)

On Thu, Nov 5, 2015 at 2:11 PM, Jonathan Wakely wrote: > On 5 November 2015 at 20:52, Daniel Gutson wrote: >> Real use cases: statistics and logging. It's a (one time) callback >> reporting that something went wrong, >> but not intended to fix things e.g. by attempting to

Re: [PING 2] [PATCH] c++/67942 - diagnose placement new buffer overflow

OK, thanks. Jason

Re: [PING] Re: [PATCH] c++/67913, 67917 - fix new expression with wrong number of elements

On 11/02/2015 09:55 PM, Jason Merrill wrote: On 10/26/2015 10:06 PM, Martin Sebor wrote: + if (TREE_CONSTANT (maybe_constant_value (outer_nelts))) +{ + if (tree_int_cst_lt (max_outer_nelts_tree, outer_nelts)) maybe_constant_value may return a constant, but that doesn't mean that

Re: [PATCH] clarify documentation of -Q --help=optimizers

On Thu, 5 Nov 2015, Martin Sebor wrote: > > There are several options marked Optimization that are actually matters of > > language semantics that I don't think have anything to do with -O options, > > e.g. -fexceptions and -fcx-limited-range. The Optimization tag really > > just means that it's

Re: [hsa 3/12] HSA libgomp plugin

This new file should have the standard libgomp copyright / license notice. -- Joseph S. Myers jos...@codesourcery.com

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

On Thu, 5 Nov 2015, Martin Jambor wrote: > libgomp plugin to be built. Because the plugin needs to use HSA > run-time library, I have introduced options --with-hsa-runtime (and > more precise --with-hsa-include and --with-hsa-lib) to help find it. New configure options should be documented in

Re: [PATCH 3/4][AArch64] Add scheduling model for Exynos M1

2015-11-05 Evandro Menezes gcc/ * config/aarch64/aarch64-cores.def: Use the Exynos M1 sched model. * config/aarch64/aarch64.md: Include "exynos-m1.md". * config/arm/arm-cores.def: Use the Exynos M1 sched model. * config/arm/arm.md:

Re: [PATCH] clarify documentation of -Q --help=optimizers

On 11/05/2015 10:09 AM, Segher Boessenkool wrote: On Thu, Nov 05, 2015 at 08:58:25AM -0700, Martin Sebor wrote: I don't think that reiterating in a condensed form what the manual doesn't make clear in many more words will help. First, even users who do find the relevant text in the manual often

Merge of HSA branch

Hi, we have had a few last-minute issues, so I apologize this is happening a bit late, but I would like to start the process or merging the HSA branch to GCC trunk. Changes to different parts of the compiler and libgomp are posted as individual emails in this thread. No two patches touch any

[hsa 6/12] IPA-HSA pass

Hi, when a target construct is gridified, the HSA GPU function is associated with the CPU function throughout the compilation, so that they can be registered as a pair in libgomp. When a target or a parallel construct is not gridified, its body emerges out of OMP expansion as one gimple

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

The new options need to be documented in invoke.texi. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH 3/4][AArch64] Add scheduling model for Exynos M1

2015-11-05 Evandro Menezes gcc/ * config/aarch64/aarch64-cores.def: Use the Exynos M1 sched model. * config/aarch64/aarch64.md: Include "exynos-m1.md". * config/arm/arm-cores.def: Use the Exynos M1 sched model. * config/arm/arm.md:

Re: [PATCH] clarify documentation of -Q --help=optimizers

On Thu, Nov 05, 2015 at 02:04:47PM -0700, Martin Sebor wrote: > On 11/05/2015 10:09 AM, Segher Boessenkool wrote: > >On Thu, Nov 05, 2015 at 08:58:25AM -0700, Martin Sebor wrote: > >>I don't think that reiterating in a condensed form what the manual > >>doesn't make clear in many more words will

Re: [PING 2] [PATCH] c++/67942 - diagnose placement new buffer overflow

On 11/05/2015 11:13 AM, Jason Merrill wrote: OK, thanks. I'm afraid the last patch that I just committed breaks libstdc++ bootstrap with the following error: /home/msebor/scm/fsf/gcc-svn/libstdc++-v3/src/c++98/ios_init.cc: In static member function ‘static bool

[PATCH 4/4][AArch64] Add cost model for Exynos M1

2015-10-25 Evandro Menezes gcc/ * config/aarch64/aarch64-cores.def: Use the Exynos M1 cost model. * config/aarch64/aarch64.c (exynosm1_addrcost_table): New variable. (exynosm1_regmove_cost): Likewise. (exynosm1_vector_cost): Likewise.

[hsa 2/12] Modifications to libgomp proper

Hi, The patch below contains all changes to libgomp files. First, it adds a new constant identifying HSA devices and a structure that is shared between libgomp and the compiler when kernels from kernels are invoked via dynamic parallelism. Second it modifies the GOMP_target_41 function so that

[hsa 12/12] HSA register allocator

Hi, because HSA backend is not based on RTL,we need our own, and it is in this patch. The allocator has been written by Michael Matz and I have put it into a separate email so that I can add him to CC, because he is much better suited to answer any questions or review comments. Thanks, Martin

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

Hi, this patch contains changes to the configuration mechanism and offload bits, so that users can build compilers with HSA support and it plays nicely with other accelerators despite using an altogether different implementation approach. With this patch, the user can request HSA support by

[hsa 9/12] Small alloc-pool fix

Hi, we use C++ new operators based on alloc-pools a lot in the subsequent patches and realized that on the current trunk, such new operators would needlessly call the placement ::new operator within the allocate method of pool-alloc. Fixed below by providing a new allocation method which does

[hsa 8/12] Pass manager changes

Hi, the following pathch has been actually already committed to trunk by Martin yesterday, but is necessary if you have an older trunk. It allows an optimization pass to declare the functon finished by returning TODO_discard_function. The pass manager will then discard the function. Martin

[hsa 4/12] OpenMP lowering/expansion changes (gridification)

Hi, the patch in this email contains the changes to make our OpenMP lowering and expansion machinery produce GPU kernels for a certain limited class of loops. The plan is to make that class quite a big bigger, but only the following is ready for submission now. Basically, whenever the compiler

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

Hi, the following small part of the merge deals with new options. It adds four independent things: 1) flag_disable_hsa is used by code in opts.c (in the first patch) to remember whether HSA has been explicitely disabled on the compiler command line. 2) -Whsa is a new warning we emit

[hsa 7/12] Disabling the vectorizer for GPU kernels/functions

Hi, in the previous email I wrote we need to "change behavior" of a few optimization passes. One was the flattening of GPU functions and the other two are in the patch below. It all comes to that, at the moment, we need to switch off the vectorizer (only for the GPU functions, of course). We

[PATCH 2/4][AArch64] Increase the loop peeling limit

2015-11-05 Evandro Menezes gcc/ * config/aarch64/aarch64.c (aarch64_override_options_internal): Increase loop peeling limit. This patch increases the limit for the number of peeled insns. With this change, I noticed no major regression in either

[PATCH] enable loop fusion on isl-15

* graphite-optimize-isl.c (optimize_isl): Call isl_options_set_schedule_maximize_band_depth. * gcc.dg/graphite/fuse-1.c: New. * gcc.dg/graphite/fuse-2.c: New. * gcc.dg/graphite/interchange-13.c: Remove bogus check. --- gcc/graphite-optimize-isl.c

[hsa 3/12] HSA libgomp plugin

Hi, the patch below adds the HSA-specific plugin for libgomp. The plugin implements the interface mandated by libgomp and takes care of finding any available HSA devices, finalizing HSAIL code and running it on HSA-capable GPUs. The plugin does not really implement any data movement functions

[hsa 10/12] HSAIL BRIG description header file (hopefully not a licensing issue)

Hi, the following patch adds a BRIG (binary representation of HSAIL) representation description. It is within a single header file describing the binary structures and constants of the format. Initially, I have created the file by copying out pieces of PDF documentation but the latest version

[PATCH] [graphite] do not collect cold loops in scops

* graphite-scop-detection.c (loop_is_valid_scop): Call optimize_loop_nest_for_speed_p. --- gcc/graphite-scop-detection.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/gcc/graphite-scop-detection.c b/gcc/graphite-scop-detection.c index ae8497d..f22e9cc 100644 ---

Re: Fix 61441

Hi, > This is looking pretty close to ready to go in, but the patch also seems > to be doing several things at once and it might be easier to review if > split into separate logical pieces. For example (this is an illustration, > not necessarily saying it should be exactly this way): (1) Add >

Re: [openacc] tile, independent, default, private and firstprivate support in c/++

On Thu, Nov 05, 2015 at 06:10:49PM -0800, Cesar Philippidis wrote: > I've applied this patch to trunk. It also includes the fortran and > template changes. Note that there is a new regression in > gfortran.dg/goacc/combined_loop.f90. Basically, the gimplifier is > complaining about reduction

[ARM][PATCH, PR 68223] arm_[su]min_cmp pattern fails

The patterns arm_smin_cmp and arm_umin_cmp patterns fail if operand 0 and operand 2 are equal and both are less than operand 1. The solution is to remove the two patterns. 2015-11-06 Michael Collison

Re: [ping] Fix PR debug/66728

On Nov 5, 2015, at 4:32 AM, Richard Biener wrote: > No idea on location lists but maybe this means we should just use the > maximum supported integer mode for CONST_WIDE_INTs? Ah, yeah, that sounds like a fine idea. Below is that version. I snuck in one more

[PATCH] Merge from gomp-4_5-branch to trunk

This patch broke bootstrap on AIX. /nasfarm/edelsohn/src/src/libstdc++-v3/src/c++98/ios_init.cc: In static member function 'static bool std::ios_base::sync_with_stdio(bool)': /nasfarm/edelsohn/src/src/libstdc++-v3/src/c++98/ios_init.cc:181:59: internal compiler error: in tree_to_shwi, at

Fix ipa-polymorphic-call-info ICE

Hi, this patch fixes tripple thinko when in ipa_polymorphic_call_context::restrict_to_inner_type when dealing with an offset that is out of the range of the type considered. In this case function should return true only when type is dynamic (so there may be additional type after the known type)

  1   2   >