Re: [gomp4.5] depend nowait support for target

On Thu, Nov 12, 2015 at 11:51:33PM +0300, Ilya Verbin wrote: > I'm unable to reproduce the hang (have tried various values of > OMP_NUM_THREADS). > The testcase just aborts at (a != 50 || b != 4 || c != 20), because > a == 37, b == 12, c == 40. The hang has been with a fprintf (stderr, "...\n");

Re: [RFC, Patch]: Optimized changes in the register used inside loop for LICM and IVOPTS.

On Fri, Nov 13, 2015 at 7:31 AM, Bin.Cheng wrote: > On Fri, Nov 13, 2015 at 2:13 PM, Jeff Law wrote: >> On 10/07/2015 10:32 PM, Ajit Kumar Agarwal wrote: >> >>> >>> 0001-RFC-Patch-Optimized-changes-in-the-register-used-ins.patch >>> >>> >>> From

Re: [RFC] Remove first_pass_instance from pass_vrp

On Thu, Nov 12, 2015 at 4:33 PM, David Malcolm wrote: > On Thu, 2015-11-12 at 15:06 +0100, Richard Biener wrote: >> On Thu, Nov 12, 2015 at 3:04 PM, Richard Biener >> wrote: >> > On Thu, Nov 12, 2015 at 2:49 PM, Tom de Vries

Re: [PATCH][GCC] Make stackalign test LTO proof

On Thu, Nov 12, 2015 at 4:07 PM, Andre Vieira wrote: > Hi, > > This patch changes this testcase to make sure LTO will not optimize away > the assignment of the local array to a global variable which was introduced > to make sure stack space was made available for

PR68264: Use unordered comparisons for tree-call-cdce.c

As reported in PR 68264, tree-call-cdce.c should be using unordered comparisons for the range checks, in order to avoid raising FE_INVALID for quiet NaNs. Tested on x86_64-linux-gnu and aarch64-linux-gnu. The test failed on aarch64-linux-gnu before the patch, but it didn't on x86_64-linux-gnu

Re: Use combined_fn in tree-vrp.c

Richard Biener writes: > On Tue, Nov 10, 2015 at 1:09 AM, Bernd Schmidt wrote: >> On 11/07/2015 01:46 PM, Richard Sandiford wrote: >>> >>> @@ -3814,8 +3817,8 @@ extract_range_basic (value_range *vr, gimple *stmt) >>> break; >>>

Re: [PATCH, 4/16] Implement -foffload-alias

On Fri, 13 Nov 2015, Tom de Vries wrote: > On 13/11/15 09:46, Richard Biener wrote: > > On Thu, 12 Nov 2015, Tom de Vries wrote: > > > > > On 11/11/15 12:00, Jakub Jelinek wrote: > > > > On Wed, Nov 11, 2015 at 11:51:02AM +0100, Richard Biener wrote: > > > > > > The option

Re: [PATCH] RFC: Enable graphite at -O3 -fprofile_use

On Fri, 13 Nov 2015, VandeVondele Joost wrote: > I'm all in favour of requiring isl and enabling graphite by default, but > would suggest to enable it with -Ofast instead. > > One reason is that certainly extracting testcases from a PGO build is > more difficult, and initially there will

Re: [patch] GSoC: Implement std::experimental::shared_ptr

On 13/11/15 11:09 +, Jonathan Wakely wrote: This is the other piece of work done by Fan You for the Google Summer of Code (and mentored by Tim). This implements experimental::shared_ptr from the Library Fundamentals TS, which differs from std::shared_ptr by supporting arrays, i.e.

Re: [PATCH] Fix ICE for boolean comparison

2015-11-13 14:28 GMT+03:00 Richard Biener : > On Fri, Nov 13, 2015 at 11:52 AM, Ilya Enkovich > wrote: >> 2015-11-13 13:38 GMT+03:00 Richard Biener : >>> On Thu, Nov 12, 2015 at 4:44 PM, Ilya Enkovich

[Ada] Iterable aspect for an integer type

This patch fixes a spurious error on an iterator loop over an integer type on which the Iterable aspect has been specified. Analysis of the loop uses the base type to find the required primitive operations, but the signature of the First primitive and others uses the first subtype instead.

Re: [PATCH, 4/16] Implement -foffload-alias

On Fri, Nov 13, 2015 at 12:29:51PM +0100, Richard Biener wrote: > > thanks for the explanation. Filed as PR68331 - '[meta-bug] fipa-pta issues'. > > > > Any feedback on the '#pragma GCC offload-alias=' bit > > above? > > Is that sort of what you had in mind? > > Yes. Whether

Re: [PATCH][ARM] PR 49526: Add support for smmul,smmla,smmls instructions

Ping. https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00686.html Thanks, Kyrill On 06/11/15 17:05, Kyrill Tkachov wrote: Hi all, This patch introduces support for the smmul, smmla and smmls instructions that appear in armv6 architecture levels and higher. To quote the SMMUL description from

Re: [PATCH][ARM] PR 68143 Properly update memory offsets when expanding setmem

Ping. https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00581.html Thanks, Kyrill On 06/11/15 10:46, Kyrill Tkachov wrote: Hi all, In this wrong-code PR the vector setmem expansion and arm_block_set_aligned_vect in particular use the wrong offset when calling adjust_automodify_address. In the

[PATCH] Fix PR68306

This fixes more of PR68306. Digging into the details reveals that we only need to check a single (but the correct one) DR and that cost modeling also gets this wrong. Thus fixed. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-11-13 Richard Biener

[PATCH] Fix PR ipa/68311

Hello. Following patch fixes PR68311, can regbootstrap on x86_64-linux-gnu. Ready for trunk? Thanks, Martin >From bc07c0709f0601e18b7ea7dfad867a5296378640 Mon Sep 17 00:00:00 2001 From: marxin Date: Thu, 12 Nov 2015 16:17:52 +0100 Subject: [PATCH] Fix PR ipa/68311

Re: [PATCH] Fix PR ipa/68035

On 11/12/2015 07:40 PM, Jan Hubicka wrote: + + /* Initialize hash values if we are not in LTO mode. */ + if (!in_lto_p) + item->get_hash (); } >>> >>> Hmm, what is the difference to the LTO mode here. I would have expected >>> that all the items >>> was

Re: [PATCH] Fix PR ipa/68311

On Fri, Nov 13, 2015 at 1:04 PM, Martin Liška wrote: > Hello. > > Following patch fixes PR68311, can regbootstrap on x86_64-linux-gnu. > > Ready for trunk? Please use auto_vec newclasses; as it gets you a stack allocation. Also use quick_push as you know the vector is large

Re: [PATCH] [ARM] neon-testgen.ml typo

On 13/11/15 11:18, Ramana Radhakrishnan wrote: Hmm. I hadn't noticed that the crypto intrinsics tests where generated by neon-testgen.ml, I thought they were hand-written. The tests I added do not cover the crypto intrinsics, so I'm going to revert r230274 and restore all the tests generated by

Re: [PATCH] Fix ICE for boolean comparison

On Fri, Nov 13, 2015 at 11:52 AM, Ilya Enkovich wrote: > 2015-11-13 13:38 GMT+03:00 Richard Biener : >> On Thu, Nov 12, 2015 at 4:44 PM, Ilya Enkovich >> wrote: >>> Hi, >>> >>> Currently compiler may ICE when loaded

[PATCH 05/N] Fix memory leaks in graphite

Hello. Patch survives regbootstrap on x86_64-linux-gnu. Ready for trunk? Thanks, Martin >From 3f84b19e0ea7eacf26a566d3ef796397dafe76ce Mon Sep 17 00:00:00 2001 From: marxin Date: Thu, 12 Nov 2015 15:45:38 +0100 Subject: [PATCH] Fix memory leaks in graphite ---

Re: [PATCH 4/4] [ARM] Add attribute/pragma target fpu=

Hi Christian, On 12/11/15 14:54, Christian Bruel wrote: Hi Kyril, ... The parts in this patch look ok to me. However, I think we need some more functionality In aarch64 we support compiling a file with no simd, including arm_neon.h and using arm_neon.h intrinsics within functions tagged with

Re: [PATCH, VECTOR ABI] Add __attribute__((__simd__)) to GCC.

Hello Jakub, I've fixed all long lines, thanks! I've also fixed max_len for "simd" attribute. Tests are fixed w/ scan for SIMD-mangled routines, routines made `extern'. ChangeLog entry was updated. gcc/ * omp-low.c (pass_omp_simd_clone::gate): If target allows - call without

Re: [PATCH 0/2] Levenshtein-based suggestions (v3)

On Fri, 2015-11-13 at 07:57 +0100, Marek Polacek wrote: > Probably coming too late, sorry. > On Thu, Nov 12, 2015 at 09:08:36PM -0500, David Malcolm wrote: > > index 4335a87..eb4e1fc 100644 > > --- a/gcc/c/c-typeck.c > > +++ b/gcc/c/c-typeck.c > > @@ -47,6 +47,7 @@ along with GCC; see the file

Re: [PATCH, VECTOR ABI] Add __attribute__((__simd__)) to GCC.

On Fri, Nov 13, 2015 at 02:54:33PM +0300, Kirill Yukhin wrote: > @@ -9013,6 +9016,35 @@ handle_omp_declare_simd_attribute (tree *, tree, tree, > int, bool *) >return NULL_TREE; > } > > +/* Handle an "simd" attribute. */ /* Handle a "simd" attribute. */ instead? > +static tree >

Re: [PATCH 05/N] Fix memory leaks in graphite

On Fri, Nov 13, 2015 at 12:43 PM, Martin Liška wrote: > Hello. > > Patch survives regbootstrap on x86_64-linux-gnu. > Ready for trunk? Ok. Richard. > Thanks, > Martin

Re: [patch] libstdc++/56158 Extend valid values of iostream bitmask types

On 12/11/15 11:09 -0700, Martin Sebor wrote: On 11/12/2015 10:08 AM, Jonathan Wakely wrote: On 12/11/15 08:48 -0700, Martin Sebor wrote: On 11/11/2015 02:48 AM, Jonathan Wakely wrote: As described in the PR, we have operator~ overloads defined for enumeration types which produce values

[gomp4] Merge trunk r230255 (2015-11-12) into gomp-4_0-branch

Hi! Committed to gomp-4_0-branch in r230293: commit 679edb57a2d0826d2965ba5d61ef11df0e3b23bf Merge: 6ec2634 0ebb8b2 Author: tschwinge Date: Fri Nov 13 09:21:42 2015 + svn merge -r 230169:230255 svn+ssh://gcc.gnu.org/svn/gcc/trunk

Re: [PATCH, 4/16] Implement -foffload-alias

On Thu, 12 Nov 2015, Tom de Vries wrote: > On 11/11/15 12:00, Jakub Jelinek wrote: > > On Wed, Nov 11, 2015 at 11:51:02AM +0100, Richard Biener wrote: > > > > The option -foffload-alias=pointer instructs the compiler to assume that > > > > objects references in an offload region do not alias. > >

Re: [committed] gen-pass-instances.awk: Simplify match regexp in handle_line

Hi Tom! As I happend to see this, and have a number of older build trees around: On Thu, 12 Nov 2015 11:09:04 +0100, Tom de Vries wrote: > this patch [and another dozen of such patches before] simplifies [...] in > gen-pass-instances.awk. > > Committed to trunk as

Re: [patch] libstdc++/56158 Extend valid values of iostream bitmask types

On 13/11/15 08:39 +, Jonathan Wakely wrote: On 12/11/15 11:09 -0700, Martin Sebor wrote: On 11/12/2015 10:08 AM, Jonathan Wakely wrote: On 12/11/15 08:48 -0700, Martin Sebor wrote: On 11/11/2015 02:48 AM, Jonathan Wakely wrote: As described in the PR, we have operator~ overloads defined

Re: [PATCH 1/2] simplify-rtx: Simplify trunc of and of shiftrt

Hello! > 2015-11-09 Segher Boessenkool > > * gcc/simplify-rtx.c (simplify_truncation): Simplify TRUNCATE > of AND of [LA]SHIFTRT. Revision r230164 (the above patch) regressed: FAIL: gcc.target/alpha/pr42269-1.c scan-assembler-not addl on alpha-linux-gnu. The

Re: [PATCH] More compile-time saving in BB vectorization

On 12 November 2015 at 21:04, Christophe Lyon wrote: > On 12 November 2015 at 16:49, Andreas Schwab wrote: >> Richard Biener writes: >> >>> * tree-vectorizer.h (vect_slp_analyze_and_verify_instance_alignment): >>>

[PATCH] Avoid useless work in loop vectorization

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2015-11-13 Richard Biener * tree-vect-loop.c (vect_analyze_loop_2): Add fatal parameter. Signal fatal failure if early checks fail. (vect_analyze_loop): If

Re: [PATCH, 11/16] Update testcases after adding kernels pass group

On Thu, 12 Nov 2015, David Malcolm wrote: > On Thu, 2015-11-12 at 15:43 +0100, Richard Biener wrote: > > On Thu, Nov 12, 2015 at 3:31 PM, Tom de Vries > > wrote: > > > On 11/11/15 12:03, Richard Biener wrote: > > >> > > >> On Mon, 9 Nov 2015, Tom de Vries wrote: > > >> >

Re: [PATCH] Avoid false vector mask conversion

On Thu, Nov 12, 2015 at 5:08 PM, Ilya Enkovich wrote: > Hi, > > When we use LTO for fortran we may have a mix 32bit and 1bit scalar booleans. > It means we may have conversion of one scalar type to another which confuses > vectorizer because values with different scalar

Re: [PATCH 1/2] simplify-rtx: Simplify trunc of and of shiftrt

On Fri, Nov 13, 2015 at 3:36 PM, Segher Boessenkool wrote: > Hi! > > On Fri, Nov 13, 2015 at 11:02:55AM +0100, Uros Bizjak wrote: >> on alpha-linux-gnu. >> >> The difference starts in combine, where before the patch, we were able >> to combine insns: >> >> (insn 7 6 8

[PATCH 4/4][AArch64] Cost CCMP instruction sequences to choose better expand order

This patch adds CCMP selection based on rtx costs. This is based on Jiong's already approved patch https://gcc.gnu.org/ml/gcc-patches/2015-09/msg01434.html with some minor refactoring and the tests updated. OK for commit? ChangeLog: 2015-11-13 Jiong Wang gcc/ *

[PATCH 3/4][AArch64] Add CCMP to rtx costs

This patch adds support for rtx costing of CCMP. The cost is the same as int/FP compare, however comparisons with zero get a slightly larger cost. This means we prefer emitting compares with zero so they can be merged with ALU operations. OK for commit? ChangeLog: 2015-11-13 Wilco Dijkstra

[PATCH] Fix SH/FDPIC bad codegen with ssp enabled

The "chk_guard_add" pattern used for loading the GOT slot address for __stack_chk_guard hard-codes use of r12 as a fixed GOT register and thus is not suitable for FDPIC, where the saved initial value of r12 from function entry is what we need. I would actually prefer removing this hack entirely

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

On Fri, Nov 13, 2015 at 11:17:38AM +0100, Uros Bizjak wrote: > IMO, this is such a small code-size regression, that it should not > block the patch submission. In that case: Kyrill, the patch is okay for trunk. Thanks! > It would be nice to know, what causes the > increase (in case, this is

[nvptx] complex vector reductions

I noticed that we weren't supporting reductions of complex type, particularly complex double. I've committed this patch to add support for vector reductions. We split the complex type apart and process each half before sticking it back together again. As it happens only 32-bit shuffles

Re: [PATCH 0/2] Levenshtein-based suggestions (v3)

On Fri, Nov 13, 2015 at 07:16:08AM -0500, David Malcolm wrote: > > > + && (TREE_CODE (TREE_TYPE (field)) == RECORD_TYPE > > > + || TREE_CODE (TREE_TYPE (field)) == UNION_TYPE)) > > > > This is RECORD_OR_UNION_TYPE_P (TREE_TYPE (field)). > > I based this code on the code in lookup_field

Re: [patch] Implement std::experimental::source_location (sort of)

On 11/13/2015 08:21 AM, Martin Sebor wrote: On 11/13/2015 05:56 AM, Jonathan Wakely wrote: This is a non-conforming implementation of https://rawgit.com/cplusplus/fundamentals-ts/v2/fundamentals-ts.html#reflection.src_loc It doesn't provide any column numbers, and fails to meet the

Re: [PATCH 0/2] Levenshtein-based suggestions (v3)

On Fri, Nov 13, 2015 at 04:53:05PM +0100, Marek Polacek wrote: > On Fri, Nov 13, 2015 at 04:44:21PM +0100, Bernd Schmidt wrote: > > On 11/13/2015 04:11 PM, Marek Polacek wrote: > > >Sorry, I changed my mind. Since QUAL_UNION_TYPE is Ada-only thing and > > >we check (RECORD_TYPE || UNION_TYPE) in

Re: [PATCH] [ARM/Aarch64] add initial Qualcomm support

On Thu, Nov 12, 2015 at 8:15 AM, Ramana Radhakrishnan wrote: > This is OK to go in with a follow up to handle this cpu in t-aprofile > similar to the other cpus in there - for bonus points please deal with > the exynos core at the same time if not already done. This

Re: [PATCH] Fix memory leaks in tree-ssa-uninit.c

On 11/13/2015 05:32 PM, Jeff Law wrote: > On 11/13/2015 05:50 AM, Martin Liška wrote: >> Hello. >> >> Patch survives regbootstrap on x86_64-linux-gnu. >> Ready for trunk? >> >> Thanks, >> Martin >> >> >> 0001-Fix-memory-leaks-in-tree-ssa-uninit.c.patch >> >> >> From

Re: [patch] fix regrename pass to ensure renamings produce valid insns

On 11/06/2015 08:51 PM, Jeff Law wrote: I think the change is fine for the trunk, though I'm still curious about how the code as-is resulted in a comparison failure. I've been retesting and I think this was a case of something else triggering an random failure - the patch made it go away on

Re: [patch] update locale support fro FreeBSD

On 12/11/15 23:32 +0100, Andreas Tobler wrote: All, with the work from Jennifer Yao and John Marino we can now update the locale support on FreeBSD to the level of DragonFly. Results of this work can be found on the results list. Here my small addendum to make it work on FreeBSD. Is this

[PATCH 1/4][AArch64] Generalize CCMP support

This patch series generalizes CCMP by adding FCCMP support and enabling more optimizations. The first patch simplifies the representation of CCMP patterns by using if-then-else which closely matches real instruction semantics. As a result the existing special CC modes and functions are no longer

Re: [PATCH 0/2] Levenshtein-based suggestions (v3)

On Fri, Nov 13, 2015 at 04:56:30PM +0100, Jakub Jelinek wrote: > On Fri, Nov 13, 2015 at 04:53:05PM +0100, Marek Polacek wrote: > > On Fri, Nov 13, 2015 at 04:44:21PM +0100, Bernd Schmidt wrote: > > > I don't think so, the three codes are adjacent so we should be generating > > > "(unsigned)(code

Re: [PATCH] [ARM/Aarch64] add initial Qualcomm support

On Fri, Nov 13, 2015 at 9:02 AM, Kyrill Tkachov wrote: > Sorry to chime in late on this, but while you're at it could > you please add an xgene1 entry? Yes, I just realized that xgene1 is missing too, I rushed the patch a little too much. I will revise it to add xgene1

Re: [patch] Implement std::experimental::source_location (sort of)

On 11/13/2015 05:56 AM, Jonathan Wakely wrote: This is a non-conforming implementation of https://rawgit.com/cplusplus/fundamentals-ts/v2/fundamentals-ts.html#reflection.src_loc It doesn't provide any column numbers, and fails to meet the requirement that using current() in a NSDMI will refer

Re: [Bulk] [OpenACC 0/7] host_data construct

On Mon, Nov 02, 2015 at 06:33:39PM +, Julian Brown wrote: > Firstly, on trunk at least, use_device_ptr variables are restricted to > pointer or array types: that restriction doesn't exist in OpenACC, nor > actually could I find it in the OpenMP 4.1 document (my guess is the > standards are

Re: TR1 Special Math

On 25 October 2015 at 20:48, Jonathan Wakely wrote: > On 25 October 2015 at 17:46, Ed Smith-Rowland <3dw...@verizon.net> wrote: >> On 10/24/2015 11:38 PM, Jonathan Wakely wrote: >>> >>> On 8 May 2015 at 15:05, Ed Smith-Rowland <3dw...@verizon.net> wrote: On

Re: [nvptx] complex vector reductions

On 11/13/2015 04:11 PM, Nathan Sidwell wrote: I noticed that we weren't supporting reductions of complex type, particularly complex double. I've committed this patch to add support for vector reductions. We split the complex type apart and process each half before sticking it back together

Re: [libstdc++ testsuite][patch] many locale tests only SUPPORTED on linux, start making these portable

On 11/11/15 22:12 +, Jonathan Wakely wrote: On 11/11/15 22:53 +0100, John Marino wrote: On 11/11/2015 10:51 PM, Jonathan Wakely wrote: On 16/10/15 11:21 +0200, John Marino wrote: The only significant comment was: e.g. `"de_DE" => "de_DE@ISO8859-15` should be `e.g. "de_DE" =>

Re: [PATCH 0/2] Levenshtein-based suggestions (v3)

On 11/13/2015 04:11 PM, Marek Polacek wrote: Sorry, I changed my mind. Since QUAL_UNION_TYPE is Ada-only thing and we check (RECORD_TYPE || UNION_TYPE) in a lot of places in the C FE, introducing RECORD_OR_UNION_TYPE_P everywhere would unnecessarily slow things down. I don't think so, the

Re: [PATCH] Fix memory leaks in tree-ssa-uninit.c

On 11/13/2015 05:50 AM, Martin Liška wrote: Hello. Patch survives regbootstrap on x86_64-linux-gnu. Ready for trunk? Thanks, Martin 0001-Fix-memory-leaks-in-tree-ssa-uninit.c.patch From 54851503251dee7a8bd074485db262715e628728 Mon Sep 17 00:00:00 2001 From: marxin Date:

Re: [gomp4.5] depend nowait support for target

On Fri, Nov 13, 2015 at 07:37:17PM +0300, Ilya Verbin wrote: > I don't know which interface to implement to maintain compatibility in the > future. > Anyway, currently it's impossible that a process will use the same > liboffloadmic > for 2 different offloading paths (say GCC's in exec and ICC's

Re: [PATCH] [ARM/Aarch64] add initial Qualcomm support

Hi Jim, On 13/11/15 16:59, Jim Wilson wrote: On Thu, Nov 12, 2015 at 8:15 AM, Ramana Radhakrishnan wrote: This is OK to go in with a follow up to handle this cpu in t-aprofile similar to the other cpus in there - for bonus points please deal with the exynos core at

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

On Wed, 2015-10-14 at 11:00 +0200, Richard Biener wrote: > On Tue, Oct 13, 2015 at 5:32 PM, David Malcolm wrote: > > On Thu, 2015-09-24 at 10:15 +0200, Richard Biener wrote: > >> On Thu, Sep 24, 2015 at 2:25 AM, David Malcolm wrote: > >> > On Wed,

[PATCH 2/4][AArch64] Add support for FCCMP

This patch adds support for FCCMP. This is trivial with the new CCMP representation - remove the restriction of FP in ccmp.c and add FCCMP patterns. Add a test to ensure FCCMP/FCCMPE are emitted as expected. OK for commit? ChangeLog: 2015-11-13 Wilco Dijkstra *

Re: TR1 Special Math

On 11/13/2015 10:32 AM, Jonathan Wakely wrote: On 25 October 2015 at 20:48, Jonathan Wakely wrote: On 25 October 2015 at 17:46, Ed Smith-Rowland <3dw...@verizon.net> wrote: On 10/24/2015 11:38 PM, Jonathan Wakely wrote: On 8 May 2015 at 15:05, Ed Smith-Rowland

Re: [gomp4.5] depend nowait support for target

On Fri, Nov 13, 2015 at 11:18:41AM +0100, Jakub Jelinek wrote: > For the offloading case, I actually see a problematic spot, namely that > GOMP_PLUGIN_target_task_completion could finish too early, and get the > task_lock before the thread that run the gomp_target_task_fn doing map_vars > +

Re: [PATCH 0/2] Levenshtein-based suggestions (v3)

On Fri, Nov 13, 2015 at 04:44:21PM +0100, Bernd Schmidt wrote: > On 11/13/2015 04:11 PM, Marek Polacek wrote: > >Sorry, I changed my mind. Since QUAL_UNION_TYPE is Ada-only thing and > >we check (RECORD_TYPE || UNION_TYPE) in a lot of places in the C FE, > >introducing RECORD_OR_UNION_TYPE_P

Re: [PATCH] PR/67682, break SLP groups up if only some elements match

On 10/11/15 12:51, Richard Biener wrote: >> >> Just noticing this... if we have a vectorization factor of 4 and matches >> is 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0 then this will split into 1, 1, 1, 1 >> and >> 1, 1, 0, 0, 0, ... where we know from the matches that it will again fail? >> >> Thus

Re: [gomp4.5] depend nowait support for target

On Fri, Nov 13, 2015 at 16:11:50 +0100, Jakub Jelinek wrote: > On Fri, Nov 13, 2015 at 11:18:41AM +0100, Jakub Jelinek wrote: > > For the offloading case, I actually see a problematic spot, namely that > > GOMP_PLUGIN_target_task_completion could finish too early, and get the > > task_lock before

Re: [PATCH 2/4][AArch64] Add support for FCCMP

Hi, Wilco. It looks good to me, but FCMP is quite different from FCCMP on Exynos M1, so it'd be helpful to have distinct types for them. Say, "fcmp{s,d}" and "fccmp{s,d}". Would it be acceptable to add this with this patch or later? Thank you. -- Evandro Menezes On 11/13/2015 10:02 AM,

[PATCH, x86] Fix posix_memalign declaration in mm_malloc.h

Followup to https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01433.html The posix_memalign declaration is incompatible with musl libc in C++, because of the exception specification (matters with -std=c++11 -pedantic-errors). It also pollutes the namespace and lacks protection against a potential

Re: [PATCH 07/N] Fix memory leaks in haifa-sched

On 11/13/2015 07:05 AM, Martin Liška wrote: Hello. Patch can bootstrap on x86_64-linux-pc and regression tests are running. Ready for trunk? Thanks, Martin 0001-Release-memory-in-haifa-sched.patch From 630eba9465d6502b49bac163f985d25aee982e03 Mon Sep 17 00:00:00 2001 From:

libcpp/C FE source range patch committed (r230331).

On Mon, 2015-11-02 at 14:14 -0500, David Malcolm wrote: > On Fri, 2015-10-30 at 00:15 -0600, Jeff Law wrote: > > On 10/23/2015 02:41 PM, David Malcolm wrote: > > > As in the previous version of this patch > > > "Implement tree expression tracking in C FE (v2)" > > > the patch now captures ranges

Fix openacc testcase

I've committed this trunk patch. This test case's loop is not parallelizeable, as it increments a (non-reduction) variable. It thus must be marked 'seq'. nathan 2015-11-13 Nathan Sidwell * testsuite/libgomp.oacc-c-c++-common/collapse-2.c: Sequential loop is

Re: [AArch64][TLSGD][2/2] Implement TLS GD traditional for tiny code model

On 05/11/15 14:57, Jiong Wang wrote: Marcus Shawcroft writes: +#ifdef HAVE_AS_TINY_TLSGD_RELOCS + return SYMBOL_TINY_TLSGD; +#else + return SYMBOL_SMALL_TLSGD; +#endif Rather than introduce blocks of conditional compilation it is better to gate different behaviours with a test on a

[PATCH v2] [ARM] PR61551 RFC: Improve costs for NEON addressing modes

Hi Following on from previous discussion: https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03464.html and IRC. I'm going to try once more to make the case for fixing the worst problem for GCC 6, pending a rewrite of the address_cost infrastructure for GCC 7. I think the rewrite you're describing is

Re: [Patch,tree-optimization]: Add new path Splitting pass on tree ssa representation

On 11/13/2015 03:13 AM, Richard Biener wrote: diff --git a/gcc/Makefile.in b/gcc/Makefile.in index 34d2356..6613e83 100644 --- a/gcc/Makefile.in +++ b/gcc/Makefile.in @@ -1474,6 +1474,7 @@ OBJS = \ tree-ssa-loop.o \ tree-ssa-math-opts.o \ tree-ssa-operands.o \ +

[patch] Define std::experimental::randint etc.

Another piece of the Library Fundamentals v2 TS, as specified by https://rawgit.com/cplusplus/fundamentals-ts/v2/fundamentals-ts.html#rand.util.randint Tested powerpc64le-linux, committed to trunk. commit 3a59dc1453e5d273e6943a0928fc7722488ae654 Author: Jonathan Wakely

Re: [PATCH] Fix PR56118

On 10/11/15 09:34, Richard Biener wrote: The following fixes PR56118 by adjusting the cost model handling of basic-block vectorization to favor the vectorized version in case estimated cost is the same as the estimated cost of the scalar version. This makes sense because we over-estimate the

[hsa] Pass kernel launch attributes through a device-specific argument

Hi, this hsa patch is analogous to the for-trunk RFC I have sent a while ago and implements passing HSA-specific grid sizes through a device-specific argument. Committed to the branch. Thanks, Martin 2015-11-13 Martin Jambor include/ * gomp-constants.h

Re: [PATCH] Fix memory leaks in tree-ssa-uninit.c

On 11/13/2015 09:58 AM, Martin Liška wrote: On 11/13/2015 05:32 PM, Jeff Law wrote: On 11/13/2015 05:50 AM, Martin Liška wrote: Hello. Patch survives regbootstrap on x86_64-linux-gnu. Ready for trunk? Thanks, Martin 0001-Fix-memory-leaks-in-tree-ssa-uninit.c.patch From

Re: [PATCH] [ARM/Aarch64] add initial Qualcomm support

Revised patch with the also missing xgene1 part added. Jim 2015-11-13 Jim Wilson * gcc/config/arm/t-aprofile (MULTILIB_MATCHES): Add lines for exynos-m1 and qdf24xx and xgene1 to match -march=armv8-a. Index: gcc/config/arm/t-aprofile

Re: [Patch,tree-optimization]: Add new path Splitting pass on tree ssa representation

On November 13, 2015 5:26:01 PM GMT+01:00, Jeff Law wrote: >On 11/13/2015 03:13 AM, Richard Biener wrote: > >>> diff --git a/gcc/Makefile.in b/gcc/Makefile.in >>> index 34d2356..6613e83 100644 >>> --- a/gcc/Makefile.in >>> +++ b/gcc/Makefile.in >>> @@ -1474,6 +1474,7 @@ OBJS = \

[PATCH] 21_strings/basic_string/capacity/wchar_t/18654.cc

http://www.cplusplus.com/reference/string/basic_string/reserve/ "Note that the resulting string capacity may be equal or greater than n." The current testcase verifies that the capacity is exactly equal to the length of the string or reserve value, but the standard allows the capacity to be

Re: [PATCH] PR fortran/68319 -- Implement checks for F2008:C1206

Hi Steve, I saw the thread on clf. That's a pretty quick turn around! OK for trunk. Thanks for the patch Paul On 13 November 2015 at 19:38, Steve Kargl wrote: > The attached patch implements the checks required by > constraint C1206 from Fortran 2008

[gomp4.5] Fix up doacross with dynamic scheduling

Hi! Guided scheduling doesn't ensure that all assigned chunks have sizes of multiple of chunk_size, chunk_size for guided scheduling actually means just minimum size of the chunk. That means we can't divide the IV or number of iterations by chunk size though, because the precondition for that is

Re: [gomp4.5] depend nowait support for target

On Fri, Nov 13, 2015 at 17:41:53 +0100, Jakub Jelinek wrote: > On Fri, Nov 13, 2015 at 07:37:17PM +0300, Ilya Verbin wrote: > > I don't know which interface to implement to maintain compatibility in the > > future. > > Anyway, currently it's impossible that a process will use the same > >

[PATCH] PR fortran/68319 -- Implement checks for F2008:C1206

The attached patch implements the checks required by constraint C1206 from Fortran 2008 standard. Built and regression tested on x86_64-*-freebsd. OK to commmit? 2015-11-13 Steven G. Kargl PR fortran/68319 * decl.c (gfc_match_data, gfc_match_entry):

RE: [PATCH 2/4][AArch64] Add support for FCCMP

> Evandro Menezes wrote: > Hi, Wilco. > > It looks good to me, but FCMP is quite different from FCCMP on Exynos M1, > so it'd be helpful to have distinct types for them. Say, "fcmp{s,d}" > and "fccmp{s,d}". Would it be acceptable to add this with this patch or > later? It would be easy to add

[gomp4.5] Fix up ordered threads simd

Hi! This fixes #pragma omp ordered threads simd expansion. In that case we want GOMP_ordered_start () / GOMP_ordered_end () calls around the block, but those calls really should be done just once, while the other stuff in between GOMP_SIMD_ORDERED_{START,END} internal calls should be expanded in

[C PATCH] Use RECORD_OR_UNION_TYPE_P macro

As promised & discussed in the thread here: . diffstat shows: c/c-decl.c | 38 +-- c/c-typeck.c | 81 +++ 2 files changed, 40 insertions(+), 79

Re: [PATCH] Fix PR68306

On Fri, Nov 13, 2015 at 2:15 PM, Uros Bizjak wrote: >> 2015-11-13 Richard Biener >> >> PR tree-optimization/68306 >> * tree-vect-data-refs.c (verify_data_ref_alignment): Move >> loop related checks ... >> (vect_verify_datarefs_alignment): ... here. >>

[RFC] Device-specific OpenMP target arguments

Hello, the patch below is is an untested trunk-only implementation of device specific GOMP_target_ext arguments, which was proposed to me by Jakub today on IRC. I'm sending this patch to make sure I understood the details well. Nevertheless, I will be commiting a tested and working version to

Re: [PATCH] Preserve the original program while using graphite

On 11/11/15 23:54, Aditya Kumar wrote: Earlier, graphite used to translate portions of the original program after scop-detection in order to represent the SCoP into polyhedral model. This was required because each basic block was represented as independent basic block in the polyhedral model.

[PATCH][AArch64] Handle function literal pools according to function size

[AArch64] Handle function literal pools according to function size gcc/ PR target/63304 * config/aarch64/aarch64-protos.h (aarch64_nopcrelative_literal_loads): Move to module scope in "aarch64.c". (aarch64_may_load_literal_pcrel) New function. *

Re: [PATCH 2/4][AArch64] Add support for FCCMP

On 11/13/2015 11:36 AM, Wilco Dijkstra wrote: Evandro Menezes wrote: Hi, Wilco. It looks good to me, but FCMP is quite different from FCCMP on Exynos M1, so it'd be helpful to have distinct types for them. Say, "fcmp{s,d}" and "fccmp{s,d}". Would it be acceptable to add this with this patch or

[PATCH] C++ FE: offer suggestions for misspelled field names

This is analogous to: "[PATCH 2/2] C FE: suggest corrections for misspelled field names" https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03380.html but for the C++ frontend. OK for trunk if it passes bootstrap? gcc/c/ChangeLog: * c-typeck.c (lookup_field_fuzzy): Move determination

Re: Automatic openacc loop partitioning

On 11/13/15 16:40, Bernd Schmidt wrote: + this_mask = (this_mask & -this_mask); Unnecessary parens. + if (!this_mask && noisy) +warning_at (loop->loc, 0, +"insufficient partitioning available to parallelize loop"); Should this really be an unconditional warning?

Re: [PATCH 2/4][AArch64] Add support for FCCMP

On Sat, Nov 14, 2015 at 1:36 AM, Wilco Dijkstra wrote: >> Evandro Menezes wrote: >> Hi, Wilco. >> >> It looks good to me, but FCMP is quite different from FCCMP on Exynos M1, >> so it'd be helpful to have distinct types for them. Say, "fcmp{s,d}" >> and "fccmp{s,d}".

Re: [PATCH] PR fortran/67803 -- Check CHARACTER array constructor element types

On Fri, Nov 13, 2015 at 01:57:13PM -0800, Steve Kargl wrote: > The attached patch fixes an ICE that occurs in arith.c(gfc_arith_concat) > because op1 and op2 have incompatible typespecs. The fix is actually > implemented in array.c(gfc_match_array_constructor) where the types > of the elements in

Re: [PATCH applied], Power9 patches #6-8 (IEEE 128-bit h/w, 128-bit direct move, integer mult/add)

Here is the combo patch for patches #6, #7, and #8 that was applied. I redid the code attributes to have a attribute for both fix and float patterns David requested. [gcc] 2015-11-13 Michael Meissner * config/rs6000/constraints.md (we constraint): New

  1   2   >