[Bug c++/68001] [cilkplus] ICE in cp_gimplify_expr, at cp/cp-gimplify.c:760

2015-11-23 Thread kyukhin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68001 --- Comment #1 from Kirill Yukhin --- Author: kyukhin Date: Mon Nov 23 12:58:12 2015 New Revision: 230755 URL: https://gcc.gnu.org/viewcvs?rev=230755=gcc=rev Log: PR c++/68001 gcc/c-family PR c++/68001 * c-gimplify.c

Re: [PATCH] Avoid false vector mask conversion

2015-11-23 Thread Ilya Enkovich
Ping 2015-11-13 16:17 GMT+03:00 Ilya Enkovich : > 2015-11-13 13:03 GMT+03:00 Richard Biener : >> On Thu, Nov 12, 2015 at 5:08 PM, Ilya Enkovich >> wrote: >>> Hi, >>> >>> When we use LTO for fortran we may have a mix

[Bug c/68499] Unclear STDC FP_CONTRACT behavior in non-standard modes

2015-11-23 Thread joseph at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68499 --- Comment #2 from joseph at codesourcery dot com --- Unknown pragmas are diagnosed with -Wunknown-pragmas (part of -Wall).

Re: Fix lto-symtab ICE during Ada LTO bootstrap

2015-11-23 Thread Eric Botcazou
> No, Interfaces.C.Extensions is non portable, so almost no Ada code out there > is using it. As I said, existing Ada code is using System.Address all the > time, so requiring any code change in this area is just a non starter. We'd > rather require that people don't use LTO with Ada rather than

[Bug tree-optimization/68492] [6 Regression] internal compiler error: in vect_is_simple_use, at tree-vect-stmts.c:8266

2015-11-23 Thread ienkovich at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68492 Ilya Enkovich changed: What|Removed |Added CC||ienkovich at gcc dot gnu.org ---

[Bug c/63303] Pointer subtraction is broken when using -fsanitize=undefined

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63303 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug other/68498] New: Replace LOOPS_MAY_HAVE_MULTIPLE_LATCHES with LOOPS_HAVE_SINGLE_LATCH

2015-11-23 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68498 Bug ID: 68498 Summary: Replace LOOPS_MAY_HAVE_MULTIPLE_LATCHES with LOOPS_HAVE_SINGLE_LATCH Product: gcc Version: 6.0 Status: UNCONFIRMED Severity: enhancement

Re: [PATCH 1/2] Libsanitizer merge from upstream r253555.

2015-11-23 Thread Maxim Ostapenko
+ Adhemerval Christophe, it looks like your kernel headers (asm/ptrace.h) don't contain ARM_VFPREGS_SIZE. Do you use old kernel version? -Maxim On 23/11/15 15:16, Christophe Lyon wrote: On 23 November 2015 at 09:07, Jakub Jelinek wrote: On Mon, Nov 23, 2015 at 10:46:33AM

Re: [OpenACC] declare directive

2015-11-23 Thread Thomas Schwinge
Hi Jim! A few things I noticed when working on merging your trunk r230275 into gomp-4_0-branch. Please fix these (on trunk). | --- gcc/c-family/c-pragma.h | +++ gcc/c-family/c-pragma.h | @@ -176,7 +178,8 @@ enum pragma_omp_clause { |PRAGMA_OACC_CLAUSE_FIRSTPRIVATE =

[Bug target/68484] _mm_storel_epi64((__m128i *)x, m); does nothing if "x" is a "volatile" ptr

2015-11-23 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68484 --- Comment #5 from H.J. Lu --- (In reply to Richard Biener from comment #4) > As the summary mentions 'volatile' I'll also point to the implementation of > the intrinsics which have > > /* Store four SPFP values. The address must be 16-byte

[Bug other/68500] Remove in_loop_pipeline usage

2015-11-23 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68500 --- Comment #1 from vries at gcc dot gnu.org --- Created attachment 36808 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36808=edit Tentative patch: "Don't reapply loops flags if unnecessary in loop_optimizer_init"

[Bug target/68484] _mm_storel_epi64((__m128i *)x, m); does nothing if "x" is a "volatile" ptr

2015-11-23 Thread glisse at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68484 --- Comment #6 from Marc Glisse --- clang has: static __inline__ void __DEFAULT_FN_ATTRS _mm_storel_epi64(__m128i *__p, __m128i __a) { struct __mm_storel_epi64_struct { long long __u; } __attribute__((__packed__, __may_alias__));

Re: [RFC] Cse reducing performance of register allocation with -O2

2015-11-23 Thread Dominik Vogt
On Tue, Oct 13, 2015 at 11:06:48AM -0600, Jeff Law wrote: > On 10/13/2015 07:12 AM, Dominik Vogt wrote: > >In some cases, the work of the cse1 pass is counterproductive, as > >we noticed on s390x. The effect described below is present since > >at least 4.8.0. Note that this may not become

[Bug plugins/54959] current_pass == NULL during invocation of pass->gate within execute_ipa_summary_passes()

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54959 Richard Biener changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Patch] PR68137, drop constant overflow flag in adjust_range_with_scev when possible

2015-11-23 Thread Richard Biener
On Mon, 23 Nov 2015, Jiong Wang wrote: > As reported by pr68137 and pr68326, r230150 caused new issues. > > Those ICEs are caused by adjust_range_with_scev getting range with > overflowed constants min or max. So given there are too many places to > generate OVF, we do a check in

Re: [PATCH 1/2] Libsanitizer merge from upstream r253555.

2015-11-23 Thread Christophe Lyon
On 23 November 2015 at 09:07, Jakub Jelinek wrote: > On Mon, Nov 23, 2015 at 10:46:33AM +0300, Maxim Ostapenko wrote: >> Index: libsanitizer/configure.ac >> === >> --- libsanitizer/configure.ac (revision 230597) >>

[Bug c/63303] Pointer subtraction is broken when using -fsanitize=undefined

2015-11-23 Thread ch3root at openwall dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63303 --- Comment #16 from Alexander Cherepanov --- On 2015-11-23 14:58, rguenth at gcc dot gnu.org wrote: > Note that in practice it needs exposal of the address constant to trigger the > bogus optimization. No. The program: #include #include

[Bug other/68500] Remove in_loop_pipeline usage

2015-11-23 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68500 --- Comment #3 from vries at gcc dot gnu.org --- Created attachment 36810 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36810=edit Tentative patch: Add PROP_loops_normal_re_lcssa

[Bug other/68500] Remove in_loop_pipeline usage

2015-11-23 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68500 --- Comment #2 from vries at gcc dot gnu.org --- Created attachment 36809 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36809=edit Tentative patch: Add PROP_scev

[Bug c/20785] Pragma STDC * (C99 FP) unimplemented

2015-11-23 Thread vincent-gcc at vinc17 dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=20785 Vincent Lefèvre changed: What|Removed |Added CC||vincent-gcc at vinc17 dot net ---

[Bug other/68500] Remove in_loop_pipeline usage

2015-11-23 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68500 --- Comment #4 from vries at gcc dot gnu.org --- First patch posted here: https://gcc.gnu.org/ml/gcc-patches/2015-11/msg02634.html Last two patches fyi-posted here: https://gcc.gnu.org/ml/gcc-patches/2015-11/msg02688.html

Re: [PATCH 1/2] Libsanitizer merge from upstream r253555.

2015-11-23 Thread Jakub Jelinek
On Mon, Nov 23, 2015 at 03:33:57PM +0300, Maxim Ostapenko wrote: > + Adhemerval > > Christophe, it looks like your kernel headers (asm/ptrace.h) don't contain > ARM_VFPREGS_SIZE. Do you use old kernel version? Unlike LLVM, we do care to support older kernel headers. So, if it is say a define,

Re: [PATCH 1/2] Libsanitizer merge from upstream r253555.

2015-11-23 Thread Christophe Lyon
On 23 November 2015 at 13:41, Jakub Jelinek wrote: > On Mon, Nov 23, 2015 at 03:33:57PM +0300, Maxim Ostapenko wrote: >> + Adhemerval >> >> Christophe, it looks like your kernel headers (asm/ptrace.h) don't contain >> ARM_VFPREGS_SIZE. Do you use old kernel version? > Yes, I do

[PATCH, C++] Wrap OpenACC wait in EXPR_STMT

2015-11-23 Thread Chung-Lin Tang
The OpenACC wait directive is represented as a call to the runtime function "GOACC_wait" instead of a tree code. I am seeing when '#pragma acc wait' is using inside a template function, the CALL_EXPR to GOACC_wait is being silently ignored/removed during tsubst_expr(). I think the correct way to

[Bug middle-end/56956] [4.9 Regression] ftrapv traps on valid abs-like code

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56956 Richard Biener changed: What|Removed |Added Known to work||5.3.0 Summary|[4.9/5

[Bug middle-end/56956] [4.9 Regression] ftrapv traps on valid abs-like code

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56956 Richard Biener changed: What|Removed |Added Known to work||5.3.0 Summary|[4.9/5

[Bug c++/68495] Error when expanding nontype variadic argument in trailing return type

2015-11-23 Thread sneves at dei dot uc.pt
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68495 --- Comment #1 from Samuel Neves --- Minimal example can be further reduced to template struct int_seq {}; constexpr struct { constexpr int operator()(int x) const { return x + 1; } } f1 {}; template auto f2(F f, int_seq) ->

[Bug go/68496] New: [libgo] reflect test fails on Linux x86-64

2015-11-23 Thread ismail at i10z dot com
-system-libunwind --with-tune=corei7-avx --with-build-config=bootstrap-lto --disable-multilib --disable-werror --disable-nls --with-fpmath=sse --enable-clocale=gnu Thread model: posix gcc version 6.0.0 20151123 (experimental) (GCC)

[Bug target/68482] No vectorization for x86-64

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68482 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/68482] No vectorization for x86-64

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68482 --- Comment #4 from Richard Biener --- Author: rguenth Date: Mon Nov 23 09:11:00 2015 New Revision: 230740 URL: https://gcc.gnu.org/viewcvs?rev=230740=gcc=rev Log: 2015-11-23 Richard Biener PR

[PATCH] Add testcase for PR68482

2015-11-23 Thread Richard Biener
Committed. Richard. 2015-11-23 Richard Biener PR tree-optimization/68482 * gcc.dg/vect/pr68482.c: New testcase. Index: gcc/testsuite/gcc.dg/vect/pr68482.c === ---

Re: [Aarch64] Use vector wide add for mixed-mode adds

2015-11-23 Thread James Greenhalgh
On Sun, Nov 22, 2015 at 06:24:19PM -0700, Michael Collison wrote: > > > On 11/22/2015 8:48 AM, James Greenhalgh wrote: > >On Sun, Nov 08, 2015 at 11:51:47PM -0700, Michael Collison wrote: > >>2015-11-06 Michael Collison > >> * config/aarch64/aarch64-simd.md

[Bug ipa/68470] [4.9/5/6 Regression] Internal Compiler Error observed by g++-4.9.2 and a few other versions (reported to Debian)

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68470 Richard Biener changed: What|Removed |Added CC||hubicka at gcc dot gnu.org,

Re: [RFC] Cse reducing performance of register allocation with -O2

2015-11-23 Thread Dominik Vogt
On Tue, Oct 13, 2015 at 11:06:48AM -0600, Jeff Law wrote: > On 10/13/2015 07:12 AM, Dominik Vogt wrote: > >In some cases, the work of the cse1 pass is counterproductive, as > >we noticed on s390x. The effect described below is present since > >at least 4.8.0. Note that this may not become

Re: [PATCH/RFC] C++ FE: expression ranges (v2)

2015-11-23 Thread Richard Biener
On Sat, Nov 21, 2015 at 9:21 AM, Jakub Jelinek wrote: > On Sat, Nov 21, 2015 at 02:16:49AM -0500, Jason Merrill wrote: >> On 11/19/2015 03:46 PM, Jason Merrill wrote: >> >On 11/15/2015 12:01 AM, David Malcolm wrote: >> >>As with the C frontend, there's an issue with tree nodes

Re: Fix lto-symtab ICE during Ada LTO bootstrap

2015-11-23 Thread Arnaud Charlet
> I updated the warning to actually check if the TBAA information is in > conflict > and silence warnings on allowed type transtions that are not > useless_type_conversion_p > (which is needed for Fortran, too). This is list of warnings I get which I > suppose will need to be adressed. Most if

Re: [PATCH] Check NULL loop->latch in verify_loop_structure

2015-11-23 Thread Richard Biener
On Mon, 23 Nov 2015, Tom de Vries wrote: > Hi, > > In verify_loop_structure, we stop checking the latch once we find that it's > NULL. > > This patch tries a bit harder: > - if !LOOPS_MAY_HAVE_MULTIPLE_LATCHES, we don't allow a NULL latch > - if LOOPS_MAY_HAVE_MULTIPLE_LATCHES, we check that

[Bug rtl-optimization/68432] [6 Regression] internal compiler error: in expand_insn, at optabs.c:6947

2015-11-23 Thread rsandifo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68432 --- Comment #8 from rsandifo at gcc dot gnu.org --- I have a patch series that fixes the bug but I need to make sure that it works on other targets that use the "enabled" attribute, and run it through an all-target test. I'm hoping to post the

Re: [PATCH 2/2] Libsanitizer merge from upstream r253555.

2015-11-23 Thread Jakub Jelinek
On Mon, Nov 23, 2015 at 10:48:30AM +0300, Maxim Ostapenko wrote: > Index: gcc/testsuite/c-c++-common/asan/halt_on_error-1.c > === > --- gcc/testsuite/c-c++-common/asan/halt_on_error-1.c (revision 0) > +++

[Bug tree-optimization/68445] [6 Regression] ICE: internal compiler error: in operator[], at vec.h

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68445 Richard Biener changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug tree-optimization/68445] [6 Regression] ICE: internal compiler error: in operator[], at vec.h

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68445 --- Comment #3 from Richard Biener --- Author: rguenth Date: Mon Nov 23 08:36:59 2015 New Revision: 230737 URL: https://gcc.gnu.org/viewcvs?rev=230737=gcc=rev Log: 2015-11-23 Richard Biener PR

[Bug tree-optimization/68492] [6 Regression] internal compiler error: in vect_is_simple_use, at tree-vect-stmts.c:8266

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68492 Richard Biener changed: What|Removed |Added Target||i?86-*-* Status|NEW

[Bug tree-optimization/68474] [6 Regression] ICE: in get_no_error_domain, at tree-call-cdce.c:699 with -funsafe-math-optimizations

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68474 Richard Biener changed: What|Removed |Added CC||rsandifo at gcc dot gnu.org Target

RE: [PING][Patch] Add support for IEEE-conformant versions of scalar fmin* and fmax*

2015-11-23 Thread David Sherwood
Hi, This is part 1 of a reworked version of a patch I originally submitted in August, rebased after Richard Sandiford's recent work on the internal functions. This first patch adds the internal function definitions and optabs that provide support for IEEE fmax()/fmin() functions. Later patches

[Bug c/68473] [6 Regression] ICE: in contains_point, at diagnostic-show-locus.c:340 after error

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68473 Richard Biener changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org Target

[Bug c/68462] -fno-strict-aliasing not respected

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68462 --- Comment #4 from Richard Biener --- With N_domains non-static GCC has to assume it has its address taken and thus it thinks that the stores to ->next, .lol and .size can alias it. So the issue that -fno-strict-aliasing is not "respected" is

Re: [PATCH, PR68337] Don't fold memcpy/memmove we want to instrument

2015-11-23 Thread Richard Biener
On Fri, Nov 20, 2015 at 3:30 PM, Ilya Enkovich wrote: > On 20 Nov 14:54, Richard Biener wrote: >> On Fri, Nov 20, 2015 at 2:08 PM, Ilya Enkovich >> wrote: >> > On 19 Nov 18:19, Richard Biener wrote: >> >> On November 19, 2015 6:12:30 PM GMT+01:00,

Re: [PATCH] Fix PR68067

2015-11-23 Thread Richard Biener
On Fri, 20 Nov 2015, Alan Lawrence wrote: > On 6 November 2015 at 10:39, Richard Biener wrote: > >> ../spec2000/benchspec/CINT2000/254.gap/src/polynom.c:358:11: error: > >> location > >> references block not in block tree > >> l1_279 = PHI <1(28), l1_299(33)> > > > > ^^^ > >

Re: [PATCH] Add LANG_HOOKS_EMPTY_RECORD_P for C++ empty class

2015-11-23 Thread Richard Biener
On Sat, Nov 21, 2015 at 12:46 AM, H.J. Lu wrote: > On Fri, Nov 20, 2015 at 2:17 PM, Jason Merrill wrote: >> On 11/20/2015 01:52 PM, H.J. Lu wrote: >>> >>> On Tue, Nov 17, 2015 at 4:22 AM, Richard Biener >>> wrote: On

[Bug rtl-optimization/68173] gcc takes a long time and a lot of memory with -O0 on source file with very large expression

2015-11-23 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68173 --- Comment #14 from rguenther at suse dot de --- On Fri, 20 Nov 2015, vmakarov at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68173 > > --- Comment #13 from Vladimir Makarov --- > (In reply to Richard Biener from

Re: basic asm and memory clobbers

2015-11-23 Thread Andrew Haley
On 21/11/15 12:56, David Wohlferd wrote: > So, what now? > > While I'd like to take the big step and start kicking out warnings for > non-top-level right now, that may be too bold for phase 3. A more > modest step for v6 would just provide a way to find them (maybe > something like

Re: [PATCH] Don't reapply loops flags if unnecessary in loop_optimizer_init

2015-11-23 Thread Richard Biener
On Mon, 23 Nov 2015, Tom de Vries wrote: > [ was: Re: [PATCH, 10/16] Add pass_oacc_kernels pass group in passes.def ] > > On 20/11/15 11:37, Richard Biener wrote: > > I'd rather make loop_optimizer_init do nothing > > if requested flags are already set and no fixup is needed and > > call the

[PATCH] Fix PR68445

2015-11-23 Thread Richard Biener
The following fixes PR68445, SLP permutation being confused by a permutation only using the "first" vector. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2015-11-23 Richard Biener PR tree-optimization/68445 *

Re: [ptx] partitioning optimization

2015-11-23 Thread Jakub Jelinek
On Mon, Nov 23, 2015 at 08:46:30AM +0100, Thomas Schwinge wrote: > Here, -fopenacc induces -lgomp. So, we'll either need a (dummy?) libgomp > available to link against in gcc/testsuite/, or come up with a way to do > LTO/offloading compilation without actually linking (libgomp into) the > final

[Bug fortran/68486] [6 Regression] 187.facerec in SPEC CPU 2000 failed to build

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68486 Richard Biener changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug target/68484] _mm_storel_epi64((__m128i *)x, m); does nothing if "x" is a "volatile" ptr

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68484 Richard Biener changed: What|Removed |Added Target||x86_64-*-*, i?86-*-*

Re: [PATCH] Mark by_ref mem_ref in build_receiver_ref as non-trapping

2015-11-23 Thread Jakub Jelinek
On Sat, Nov 21, 2015 at 07:34:02PM +0100, Tom de Vries wrote: > Mark by_ref mem_ref in build_receiver_ref as non-trapping > > 2015-11-21 Tom de Vries > > * omp-low.c (build_receiver_ref): Mark by_ref mem_ref as non-trapping. This is ok. > > --- > gcc/omp-low.c

[PATCH, gcc5 backport] Fix PR ipa/65908

2015-11-23 Thread Martin Liška
Hi. At the end of last week, Richi asked me to back port aforementioned PR. The patch contains two parts: first one is the patch that was applied to trunk and the second one is a hunk that implements param_used_p (coming from r222374). Patch can bootstrap and survives regression tests on

[Bug go/68477] error: type variant differs by TYPE_STRING_FLAG.

2015-11-23 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68477 Markus Trippelsdorf changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug libfortran/51119] MATMUL slow for large matrices

2015-11-23 Thread Joost.VandeVondele at mat dot ethz.ch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51119 --- Comment #22 from Joost VandeVondele --- (In reply to Thomas Koenig from comment #21) > I assume that for small matrices bordering on the silly > (say, a matrix multiplication with dimensions of (1,2) and (2,1)) > the inline code will be

Re: [PATCH] GCC system.h and Graphite header order

2015-11-23 Thread Richard Biener
On Sun, Nov 22, 2015 at 4:13 AM, Sebastian Pop wrote: > On Sat, Nov 21, 2015 at 4:03 PM, David Edelsohn wrote: >> Graphite relies on the ISL library and includes multiple ISL headers. >> The ISL headers refer to identifiers that are poisoned for use in GCC.

[Bug middle-end/68221] libgomp reduction-11/12 failures

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68221 --- Comment #5 from Richard Biener --- Index: gcc/tree-ssa-sccvn.c === --- gcc/tree-ssa-sccvn.c(revision 230671) +++ gcc/tree-ssa-sccvn.c(working copy) @@ -750,8

[Bug go/66574] Time is provided in millisecond precision instead of nanoseconds as described in go documentation

2015-11-23 Thread boger at us dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66574 --- Comment #6 from boger at us dot ibm.com --- (In reply to Ian Lance Taylor from comment #5) > Fixed on mainline. Can this be backported to the gcc 5 branch?

Re: [PATCH, PR68337] Don't fold memcpy/memmove we want to instrument

2015-11-23 Thread Richard Biener
On Mon, Nov 23, 2015 at 12:33 PM, Ilya Enkovich wrote: > On 23 Nov 11:44, Richard Biener wrote: >> On Mon, Nov 23, 2015 at 11:10 AM, Ilya Enkovich >> wrote: >> > On 23 Nov 10:39, Richard Biener wrote: >> >> On Fri, Nov 20, 2015 at 3:30 PM, Ilya

Re: [AArch64][PATCH 7/7] Add NEON intrinsics vqrdmlah_lane and vqrdmlsh_lane.

2015-11-23 Thread James Greenhalgh
On Fri, Oct 23, 2015 at 01:30:46PM +0100, Matthew Wahab wrote: > The ARMv8.1 architecture extension adds two Adv.SIMD instructions, > sqrdmlah and sqrdmlsh. This patch adds the NEON intrinsics vqrdmlah_lane > and vqrdmlsh_lane for these instructions. The new intrinsics are of the > form

[Bug target/68483] [5/6 Regression] gcc 5.2: suboptimal code compared to 4.9

2015-11-23 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68483 Jakub Jelinek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[5 PATCH] Fix ICE with mangling aliases (PR c++/67354)

2015-11-23 Thread Jakub Jelinek
Hi! On Thu, Nov 19, 2015 at 03:04:35PM -0500, Jason Merrill wrote: > On 11/19/2015 07:40 AM, Jakub Jelinek wrote: > >@@ -4502,6 +4509,7 @@ c_parse_final_cleanups (void) > > > >locus_at_end_of_parsing = input_location; > >at_eof = 1; > >+ defer_mangling_aliases = false; > > Let's clear

Re: [PATCH, PR68337] Don't fold memcpy/memmove we want to instrument

2015-11-23 Thread Ilya Enkovich
On 23 Nov 14:29, Richard Biener wrote: > On Mon, Nov 23, 2015 at 12:33 PM, Ilya Enkovich > wrote: > > > > I see. But it should still be OK to check type in case of strict aliasing, > > right? > > No, memcpy is always "no-strict-aliasing" > Thanks a lot for help!

[PATCH 2/6] Fix memory leak in tree-ssa

2015-11-23 Thread marxin
gcc/ChangeLog: 2015-11-20 Martin Liska * tree-ssa.c (redirect_edge_var_map_destroy): Release vectors that are used as a second argument of a hash_map. --- gcc/tree-ssa.c | 5 + 1 file changed, 5 insertions(+) diff --git a/gcc/tree-ssa.c b/gcc/tree-ssa.c

Re: Enable pointer TBAA for LTO

2015-11-23 Thread Martin Jambor
Hi, On Mon, Nov 23, 2015 at 12:00:25AM +0100, Jan Hubicka wrote: > Hi, > here is updated patch which I finally comitted today. It addresses all the > comments > and also fixes one nasty bug that really cost me a lot of time to understand. > > + /* LTO type merging does not make any

[PATCH 0/6] Another fixes of various memory leaks

2015-11-23 Thread marxin
Hi. Following series has been just bootregtested on x86_64-linux-gnu (all patches together). Ready for trunk? Thanks, Martin marxin (6): Fix memory leak in cilk Fix memory leak in tree-ssa Fix memory leaks in IPA devirt Fix memory leak in loop_vec_info Fix parser memory leak in

[Bug ipa/65908] [5 Regression] ICE: in expand_thunk, at cgraphunit.c:1700

2015-11-23 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65908 Martin Liška changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/65908] [5 Regression] ICE: in expand_thunk, at cgraphunit.c:1700

2015-11-23 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65908 --- Comment #16 from Martin Liška --- Author: marxin Date: Mon Nov 23 13:58:40 2015 New Revision: 230756 URL: https://gcc.gnu.org/viewcvs?rev=230756=gcc=rev Log: PR ipa/65908 * ipa-icf.c (sem_item::target_supports_symbol_aliases):

[hsa] Fix operand and instruction types for bit-ops

2015-11-23 Thread Martin Jambor
Hi, I have applied the following patch tot he HSA branch to fix some rather unexpected type requirements of the HSA finalizer we discovered when running the target-3[34].c libgomp tests. Thanks, Martin 2015-11-23 Martin Jambor * hsa-gen.c

Re: [PATCH 1/2] Libsanitizer merge from upstream r253555.

2015-11-23 Thread Maxim Ostapenko
On 23/11/15 16:00, Christophe Lyon wrote: On 23 November 2015 at 13:41, Jakub Jelinek wrote: On Mon, Nov 23, 2015 at 03:33:57PM +0300, Maxim Ostapenko wrote: + Adhemerval Christophe, it looks like your kernel headers (asm/ptrace.h) don't contain ARM_VFPREGS_SIZE. Do you use

[Bug middle-end/68221] libgomp reduction-11/12 failures

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68221 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

Re: [PATCH 1/2] Libsanitizer merge from upstream r253555.

2015-11-23 Thread Jakub Jelinek
On Mon, Nov 23, 2015 at 04:21:34PM +0300, Maxim Ostapenko wrote: > Yeah, right. I've asked about kernel headers just to make sure I correctly > understand the issue. > > Actually, I see such code in > lib/sanitizer_common/sanitizer_platform_limits_posix.cc: > > #if defined(PTRACE_GETVFPREGS) &&

Re: Fix lto-symtab ICE during Ada LTO bootstrap

2015-11-23 Thread Richard Biener
On Mon, Nov 23, 2015 at 12:45 PM, Eric Botcazou wrote: >> No, Interfaces.C.Extensions is non portable, so almost no Ada code out there >> is using it. As I said, existing Ada code is using System.Address all the >> time, so requiring any code change in this area is just a

Re: [AArch64][PATCH 6/7] Add NEON intrinsics vqrdmlah and vqrdmlsh.

2015-11-23 Thread James Greenhalgh
On Fri, Oct 23, 2015 at 01:26:11PM +0100, Matthew Wahab wrote: > The ARMv8.1 architecture extension adds two Adv.SIMD instructions, > sqrdmlah and sqrdmlsh. This patch adds the NEON intrinsics vqrdmlah and > vqrdmlsh for these instructions. The new intrinsics are of the form > vqrdml{as}h[q]_. >

[Bug ipa/68470] [4.9/5/6 Regression] Internal Compiler Error observed by g++-4.9.2 and a few other versions (reported to Debian)

2015-11-23 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68470 Martin Liška changed: What|Removed |Added CC||marxin at gcc dot gnu.org --- Comment #4

[PATCH 4/6] Fix memory leak in loop_vec_info

2015-11-23 Thread marxin
gcc/ChangeLog: 2015-11-23 Martin Liska * tree-vect-loop-manip.c (vect_create_cond_for_alias_checks): Do not release memory for comp_alias_ddrs. * tree-vect-loop.c (destroy_loop_vec_info): Release the memory for all loop_vec_info. ---

[PATCH 5/6] Fix parser memory leak in cilk_simd_fn_info

2015-11-23 Thread marxin
gcc/cp/ChangeLog: 2015-11-23 Martin Liska * parser.c (cp_parser_late_parsing_cilk_simd_fn_info): Release tokens. --- gcc/cp/parser.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c index 24ed404..fd5c7ec 100644 ---

[PATCH 6/6] Fix memory leak in tree-chkp.c

2015-11-23 Thread marxin
gcc/ChangeLog: 2015-11-23 Martin Liska * tree-chkp.c (chkp_make_static_bounds): Release buffer used for string. --- gcc/tree-chkp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gcc/tree-chkp.c b/gcc/tree-chkp.c index 34d9dfc..8b6381f 100644 ---

[PATCH 1/6] Fix memory leak in cilk

2015-11-23 Thread marxin
gcc/c/ChangeLog: 2015-11-20 Martin Liska PR c++/68312 * c-array-notation.c (fix_builtin_array_notation_fn): Use release_vec_vec instead of vec::release. (build_array_notation_expr): Likewise. (fix_conditional_array_notations_1):

[PATCH 3/6] Fix memory leaks in IPA devirt

2015-11-23 Thread marxin
gcc/ChangeLog: 2015-11-20 Martin Liska * ipa-devirt.c (ipa_devirt): Use auto_vec instead of a local-scope vec. Release final_warning_records. --- gcc/ipa-devirt.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/gcc/ipa-devirt.c

[hsa] depend nowait support for target

2015-11-23 Thread Martin Jambor
On Fri, Nov 13, 2015 at 04:11:50PM +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

Re: [hsa] depend nowait support for target

2015-11-23 Thread Jakub Jelinek
On Mon, Nov 23, 2015 at 03:12:05PM +0100, Martin Jambor wrote: > +/* Thread routine to run a kernel asynchronously. */ > + > +static void * > +run_kernel_asynchronously (void *thread_arg) > +{ > + struct async_run_info *info = (struct async_run_info *) thread_arg; > + int device = info->device;

[Bug c/63326] whether a #pragma is a statement depends on the type of pragma

2015-11-23 Thread gang.chen.5i5j at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63326 --- Comment #19 from Chen Gang --- (In reply to Andrew Pinski from comment #18) > (In reply to Chen Gang from comment #17) > > I guess the diff below should be OK, I shall give a make check test. > > I would rather have the C front-end behavior

[Bug tree-optimization/68492] [6 Regression] internal compiler error: in vect_is_simple_use, at tree-vect-stmts.c:8266

2015-11-23 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68492 --- Comment #4 from H.J. Lu --- (In reply to Ilya Enkovich from comment #3) > Looks like a duplicate of PR68327. r230743 fixed 465.tonto build for me. r230743 still fails for me. Can you show me your linking command line for 465.tonto build?

[Bug c/63326] whether a #pragma is a statement depends on the type of pragma

2015-11-23 Thread gang.chen.5i5j at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63326 --- Comment #20 from Chen Gang --- (In reply to Andrew Pinski from comment #18) > (In reply to Chen Gang from comment #17) > > I guess the diff below should be OK, I shall give a make check test. > > I would rather have the C front-end behavior

Re: [AARCH64] Adding constant folding for __builtin_fmulx* with scalar 32 and 64 bit arguments

2015-11-23 Thread James Greenhalgh
On Mon, Nov 09, 2015 at 11:40:11AM +, Bilyan Borisov wrote: > This patch adds an extension to aarch64_gimple_fold_builtin () that does > constant folding on __builtin_fmulx* calls for 32 and 64 bit floating point > scalar modes. We fold when both arguments are constant, as well as when only >

[hsa] Use new format of device-specific target arguments

2015-11-23 Thread Martin Jambor
Hi, I have committed this branch which changes the format of device-specific arguments of GOMP_target_ext to something closer to what Jakub wants. Specifically, lowest 7 bits determine the accelerator type (0 means all of them), next bit determines whether the next element in args array is also

Re: [PATCH 0/6] Another fixes of various memory leaks

2015-11-23 Thread Bernd Schmidt
On 11/23/2015 02:49 PM, marxin wrote: Following series has been just bootregtested on x86_64-linux-gnu (all patches together). All ok except 5/6 which I'm not finding obvious. Better to have a cilk/c++ person have a look. In the future, a few more explanations would help with reviewing.

[hsa] Perform version checks in HSA plugin

2015-11-23 Thread Martin Jambor
Hi, the following patch against the HSA branch makes it call GOMP_offload_register_ver and GOMP_offload_unregister_ver as opposed to the routines without version information and adds a version check to the libgomp plugin along the lines other plugins do it. Committed to the branch, any feedback

Re: [hsa] Use new format of device-specific target arguments

2015-11-23 Thread Jakub Jelinek
On Mon, Nov 23, 2015 at 03:35:48PM +0100, Martin Jambor wrote: > +/* If the value is directly embeded in target argument, it should be a 16-bit > + at most and shifted by tis many bits. */ this Jakub

[Bug target/68484] _mm_storel_epi64((__m128i *)x, m); does nothing if "x" is a "volatile" ptr

2015-11-23 Thread vvsed at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68484 --- Comment #7 from Vladimir Sedach --- The "store" pointer could be not only volatile, but also static or global with same error.

[Bug c/68499] Unclear STDC FP_CONTRACT behavior in non-standard modes

2015-11-23 Thread vincent-gcc at vinc17 dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68499 Vincent Lefèvre changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug tree-optimization/68501] New: [6 Regression] sqrt builtin is not used anymore

2015-11-23 Thread afomin.mailbox at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68501 Bug ID: 68501 Summary: [6 Regression] sqrt builtin is not used anymore Product: gcc Version: 6.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug tree-optimization/66721] [6 regression] gcc.target/i386/pr61403.c FAILs

2015-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66721 --- Comment #3 from Richard Biener --- So I have a "solution" that I hope to prettify a bit still. The basic issue is that SLP is "broken" in that it builds a tree of operations instead of a graph. That is, it un-CSEs all n in the testcase:

[Bug target/68261] GCC needs to use optimized version of memcpy

2015-11-23 Thread geir at cray dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68261 --- Comment #4 from Geir Johansen --- (In reply to Andrew Pinski from comment #1) > Note glibc does provide a memcpy which is optimized for each target via > ifuncs. What version of glibc are you running. 2.11.3

<    1   2   3   4   >