Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope

2016-05-06 Thread Martin Liška
On 05/06/2016 01:48 PM, Yury Gribov wrote: > On 05/06/2016 02:04 PM, Martin Liška wrote: >> Hello. >> >> I've started working on the patch couple of month go, basically after >> a brief discussion with Jakub on IRC. >> >> I'm sending the initial version which can successfully run instrumented >>

Re: [gimplefe] Regarding command line option handling

2016-05-06 Thread Prasad Ghangal
On 6 May 2016 at 16:09, Richard Biener wrote: > On Wed, May 4, 2016 at 4:29 PM, Prasad Ghangal > wrote: >> On 4 May 2016 at 15:54, Richard Biener wrote: >>> On Wed, May 4, 2016 at 11:46 AM, Prasad Ghangal >>>

[Bug tree-optimization/70970] Misaligned SSE with auto-vectorization

2016-05-06 Thread rcc.dark at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70970 --- Comment #4 from Rodrigo --- Well, without auto-vectorization the code should work (x86 allows non-SSE misaligned reads and writes). But I think understand your rationale. Thanks.

Re: [PATCH] Make basic asm implicitly clobber memory

2016-05-06 Thread Bernd Edlinger
On 05/06/16 08:35, David Wohlferd wrote: > On 5/5/2016 10:29 AM, Bernd Edlinger wrote: >> Hi! >> >> this patch is inspired by recent discussion about basic asm: >> >> Currently a basic asm is an instruction scheduling barrier, >> but not a memory barrier, and most surprising, basic asm >> does

Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope

2016-05-06 Thread Martin Liška
On 05/06/2016 02:38 PM, Jakub Jelinek wrote: > On Fri, May 06, 2016 at 02:48:30PM +0300, Yury Gribov wrote: >>> 6) As the use-after-scope stuff is already included in libsanitizer, no >>> change is needed for the library >> >> Note that upstream seems to use a different cmdline interface. They

Re: [PATCH] Fix memory leak in tree-if-conv.c

2016-05-06 Thread Richard Biener
On Fri, May 6, 2016 at 2:40 PM, Martin Liška wrote: > On 05/03/2016 11:07 AM, Bin.Cheng wrote: >> Patch applied as suggested at r235808. >> >> Thanks, >> bin > > Hi. > > Following patch introduces memory leak: > /home/marxin/Programming/gcc2/objdir/gcc/xgcc >

[Bug tree-optimization/70948] [7 Regression] r235622 caused gcc.c-torture/execute/va-arg-pack-1.c execution failure AArch64

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70948 --- Comment #6 from Richard Biener --- Author: rguenth Date: Fri May 6 12:53:26 2016 New Revision: 235963 URL: https://gcc.gnu.org/viewcvs?rev=235963=gcc=rev Log: 2016-05-06 Richard Biener PR

[Bug tree-optimization/70948] [7 Regression] r235622 caused gcc.c-torture/execute/va-arg-pack-1.c execution failure AArch64

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70948 Richard Biener changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug fortran/70937] [7 Regression] ICE: tree code ‘ssa_name’ is not supported in LTO streams

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70937 --- Comment #13 from Richard Biener --- So it looks like we gimplify type sizes multiple times (in what order?!) and from ..__result = MAX_EXPR <*n, 0>; typedef character(kind=1) struct

[Bug c++/70977] Error during compilation of facebook/wangle (flag c++0x works, flag c++14 fails).

2016-05-06 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70977 Markus Trippelsdorf changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last reconfirmed|

[PATCH] Fix memory leak in tree-if-conv.c

2016-05-06 Thread Martin Liška
On 05/03/2016 11:07 AM, Bin.Cheng wrote: > Patch applied as suggested at r235808. > > Thanks, > bin Hi. Following patch introduces memory leak: /home/marxin/Programming/gcc2/objdir/gcc/xgcc -B/home/marxin/Programming/gcc2/objdir/gcc/-fno-diagnostics-show-caret -fdiagnostics-color=never

Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope

2016-05-06 Thread Jakub Jelinek
On Fri, May 06, 2016 at 02:48:30PM +0300, Yury Gribov wrote: > >6) As the use-after-scope stuff is already included in libsanitizer, no > >change is needed for the library > > Note that upstream seems to use a different cmdline interface. They don't > have a dedicated -fsanitize=use-after-scope

Re: [PATCH] Fix memory leak in tree-inliner

2016-05-06 Thread Martin Liška
On 05/06/2016 12:56 PM, Richard Biener wrote: > Hmmm. But this means debug stmt remapping calls > remap_dependence_clique which may end up bumping > cfun->last_clique and thus may change code generation. > > So what debug stmts contain MEM_REFs? If you put an assert > processing_debug_stmt == 0

Re: Please, take '-Wmisleading-indentation' out of -Wall

2016-05-06 Thread Andrew Haley
On 05/05/2016 07:56 PM, Antonio Diaz Diaz wrote: > Take this example http://gcc.gnu.org/ml/gcc-patches/2016-03/msg00261.html > > The user sees this: > >if (flagA) // GUARD > foo (0); // BODY > #if SOME_CONDITION_THAT_DOES_NOT_HOLD >if (flagB) > #endif > foo (1); // NEXT

[Bug c++/70979] New: [C++14] g++ falsely accepts constexpr function returning a lambda

2016-05-06 Thread felix.morgner at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70979 Bug ID: 70979 Summary: [C++14] g++ falsely accepts constexpr function returning a lambda Product: gcc Version: 5.3.0 Status: UNCONFIRMED Severity: normal

[PATCH] Fix PR70948

2016-05-06 Thread Richard Biener
The following fixes PR70948, a failure of PTA considering all fields of va_list being clobbered (assigned from NONLOCAL) for __builtin_va_start. With the new pointer-vs.-decl comparison optimization this bug manifests as a miscompile of gcc.c-torture/execute/va-arg-pack-1.c on AARCH64.

Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope

2016-05-06 Thread Jakub Jelinek
On Fri, May 06, 2016 at 01:04:30PM +0200, Martin Liška wrote: > I've started working on the patch couple of month go, basically after > a brief discussion with Jakub on IRC. > > I'm sending the initial version which can successfully run instrumented > tramp3d, postgresql server and Inkscape. It

[Bug rtl-optimization/70978] [5 Regression] internal compiler error: in assign_by_spills, at lra-assigns.c:1428 on arm

2016-05-06 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70978 ktkachov at gcc dot gnu.org changed: What|Removed |Added Known to work||4.9.4, 6.1.0, 7.0 Target

[Bug rtl-optimization/70978] New: [5 Regression] internal compiler error: in assign_by_spills, at lra-assigns.c:1428 on arm

2016-05-06 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70978 Bug ID: 70978 Summary: [5 Regression] internal compiler error: in assign_by_spills, at lra-assigns.c:1428 on arm Product: gcc Version: 5.3.1 Status: UNCONFIRMED

[Bug target/70976] Useless vectorization leads to degradation of performance

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70976 Richard Biener changed: What|Removed |Added Keywords||missed-optimization

[Bug debug/70935] [6/7 Regression] ICE: verify_ssa failed (error: definition in block 9 does not dominate use in block 12) w/ -O3 -g

2016-05-06 Thread ienkovich at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70935 --- Comment #5 from Ilya Enkovich --- Author: ienkovich Date: Fri May 6 12:07:25 2016 New Revision: 235962 URL: https://gcc.gnu.org/viewcvs?rev=235962=gcc=rev Log: gcc/ 2016-05-06 Yuri Rumyantsev PR debug/70935

[Bug c++/70977] Error during compilation of facebook/wangle (flag c++0x works, flag c++14 fails).

2016-05-06 Thread aleksandergajewski at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70977 Aleksander Gajewski changed: What|Removed |Added CC||aleksandergajewski at gmail dot

[Bug fortran/70937] [7 Regression] ICE: tree code ‘ssa_name’ is not supported in LTO streams

2016-05-06 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70937 --- Comment #12 from rguenther at suse dot de --- On Fri, 6 May 2016, Richard Biener wrote: > On Fri, 6 May 2016, dominiq at lps dot ens.fr wrote: > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70937 > > > > --- Comment #10 from Dominique

[Bug c++/70977] New: Error during compilation of facebook/wangle (flag c++0x works, flag c++14 fails).

2016-05-06 Thread aleksandergajewski at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70977 Bug ID: 70977 Summary: Error during compilation of facebook/wangle (flag c++0x works, flag c++14 fails). Product: gcc Version: 6.1.0 Status: UNCONFIRMED

Re: increase alignment of global structs in increase_alignment pass

2016-05-06 Thread Richard Biener
On Wed, 4 May 2016, Prathamesh Kulkarni wrote: > On 23 February 2016 at 21:49, Prathamesh Kulkarni > wrote: > > On 23 February 2016 at 17:31, Richard Biener wrote: > >> On Tue, 23 Feb 2016, Prathamesh Kulkarni wrote: > >> > >>> On 22 February

Re: [PATCH] Improve min/max

2016-05-06 Thread Kirill Yukhin
On 04 May 21:53, Jakub Jelinek wrote: > Hi! > > AVX512BW has EVEX insns for these. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK for trunk. -- Thanks, K > > 2016-05-04 Jakub Jelinek > > * config/i386/sse.md (*v8hi3, *v16qi3): Add >

Re: [PATCH] Improve whole vector right shift

2016-05-06 Thread Kirill Yukhin
On 04 May 21:51, Jakub Jelinek wrote: > Hi! > > In this case the situation is more complicated, because for > V*HI we need avx512bw and avx512vl, while for V*SI only avx512vl > is needed and both are in the same pattern. But we already have > a pattern that does the right thing right after the

Simple bitop reassoc in match.pd (was: Canonicalize X u< X to UNORDERED_EXPR)

2016-05-06 Thread Marc Glisse
On Tue, 3 May 2016, Richard Biener wrote: On Tue, May 3, 2016 at 3:26 PM, Marc Glisse wrote: On Tue, 3 May 2016, Richard Biener wrote: On Tue, May 3, 2016 at 8:36 AM, Marc Glisse wrote: This removes the duplication. I also removed the case

Re: [PATCH] Improve vec extraction

2016-05-06 Thread Kirill Yukhin
On 04 May 21:47, Jakub Jelinek wrote: > Hi! > > While EVEX doesn't have vextracti128, we can use vextracti32x4; > unfortunately without avx512dq we need to use full zmm input operand, > but that shouldn't be a big deal when we hardcode 1 as immediate. > > Bootstrapped/regtested on x86_64-linux

Re: [PATCH] Improve *pmaddwd

2016-05-06 Thread Kirill Yukhin
On 04 May 21:48, Jakub Jelinek wrote: > Hi! > > As the testcase shows, we unnecessarily disallow xmm16+, even when > we can use them for -mavx512bw. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK for trunk. -- Thanks, K > > 2016-05-04 Jakub Jelinek

Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope

2016-05-06 Thread Yury Gribov
On 05/06/2016 02:04 PM, Martin Liška wrote: Hello. I've started working on the patch couple of month go, basically after a brief discussion with Jakub on IRC. I'm sending the initial version which can successfully run instrumented tramp3d, postgresql server and Inkscape. It catches the basic

Re: [PATCH] Improve vec_concatv?sf*

2016-05-06 Thread Kirill Yukhin
On 04 May 21:44, Jakub Jelinek wrote: > Hi! > > Another pair of define_insns. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK for trunk. -- Thanks, K > > 2016-05-04 Jakub Jelinek > > * config/i386/sse.md (*vec_concatv2sf_sse4_1,

Re: [PATCH] Improve vec_interleave*

2016-05-06 Thread Kirill Yukhin
On 04 May 21:41, Jakub Jelinek wrote: > Hi! > > Another 3 define_insns that can handle xmm16+ operands. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK for trunk. -- Thanks, K > > 2016-05-04 Jakub Jelinek > > * config/i386/sse.md

[Bug rtl-optimization/70976] New: Useless vectorization leads to degradation of performance

2016-05-06 Thread b7.10110111 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70976 Bug ID: 70976 Summary: Useless vectorization leads to degradation of performance Product: gcc Version: 6.1.0 Status: UNCONFIRMED Severity: normal

Re: [PATCH] Improve other 13 define_insns

2016-05-06 Thread Kirill Yukhin
On 04 May 21:43, Jakub Jelinek wrote: > Hi! > > This patch tweaks more define_insns at once, again all the insns > should be already in AVX512F or AVX512VL. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK for trunk. -- Thanks, K > > 2016-05-04 Jakub Jelinek

Re: [PATCh] Improve sse_mov{hl,lh}ps

2016-05-06 Thread Kirill Yukhin
On 04 May 21:37, Jakub Jelinek wrote: > Hi! > > Another pair of define_insns where all the VEX insns have EVEX variant > in AVX512VL. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK for trunk. -- Thanks, K > > 2016-05-04 Jakub Jelinek > >

Re: [PATCH] Improve *avx_cvtp?2??256_2

2016-05-06 Thread Kirill Yukhin
On 04 May 21:35, Jakub Jelinek wrote: > Hi! > > Not sure how to easily construct a testcase for this (these insns are > usually used for vectorization, and then it really depends on register > pressure). > But in any case, looking at documentation it seems all the used insns are > available

[Bug libstdc++/70975] experimental/filesystem/operations/copy.cc FAILs on Solaris 12

2016-05-06 Thread ro at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70975 Rainer Orth changed: What|Removed |Added Target Milestone|--- |7.0

[Bug libstdc++/70975] New: experimental/filesystem/operations/copy.cc FAILs on Solaris 12

2016-05-06 Thread ro at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70975 Bug ID: 70975 Summary: experimental/filesystem/operations/copy.cc FAILs on Solaris 12 Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal

[Bug rtl-optimization/70961] Regrename ignores preferred_rename_class

2016-05-06 Thread wdijkstr at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70961 --- Comment #3 from Wilco --- (In reply to Eric Botcazou from comment #2) > Pass #2 ignores it since the preference simply couldn't be honored. In which case it should not rename that chain rather than just ignore the preference (and a

[Bug tree-optimization/70964] [7 Regression] internal compiler error: in single_succ_edge, at basic-block.h:351

2016-05-06 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70964 --- Comment #3 from ktkachov at gcc dot gnu.org --- Another testcase that started ICEing with r235842. int a, b; int fn1 (int p1) { return p1 < 0 ? p1 : a; } void fn2 () { lbl_100: b = 1; for (; b != 21; b = fn1 (b)) ; goto lbl_100;

[Bug fortran/70937] [7 Regression] ICE: tree code ‘ssa_name’ is not supported in LTO streams

2016-05-06 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70937 --- Comment #11 from rguenther at suse dot de --- On Fri, 6 May 2016, dominiq at lps dot ens.fr wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70937 > > --- Comment #10 from Dominique d'Humieres --- > With the second patch in comment 9

Re: [PATCH] Fix PR70937

2016-05-06 Thread Richard Biener
On Fri, 6 May 2016, Richard Biener wrote: > > The following patch fixes another case of missing DECL_EXPR in the FE. > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > Ok for trunk? Dominique noticed a FAIL early which is fixed by adjusting the patch to only handle

[Bug fortran/70937] [7 Regression] ICE: tree code ‘ssa_name’ is not supported in LTO streams

2016-05-06 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70937 --- Comment #10 from Dominique d'Humieres --- With the second patch in comment 9 applied on top of my working tree (heavily patched), I get [Book15] f90/bug% gfc /opt/gcc/_clean/gcc/testsuite/gfortran.dg/allocatable_function_4.f90

RE: [PATCH 3/4] Add support to run auto-vectorization tests for multiple effective targets

2016-05-06 Thread Matthew Fortune
Robert Suchanek writes: > I'm resending this patch as it has been rebased and updated. I reverted > a change to check_effective_target_vect_call_lrint procedure because it > does not use cached result. Conceptually I think this is a good idea and to the extent that I

[PATCH] Fix PR70937

2016-05-06 Thread Richard Biener
The following patch fixes another case of missing DECL_EXPR in the FE. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Ok for trunk? Thanks, Richard. 2016-05-06 Richard Biener PR fortran/70937 * trans-decl.c (gfc_trans_vla_type_sizes): Add

Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope

2016-05-06 Thread Martin Liška
Hello. One more issue I forgot to mention in the previous email: e) As one can come up with a source code which jumps to a label within a block scope (use-after-scope-goto-1.c): // { dg-do run } // { dg-additional-options "-fsanitize=use-after-scope -fstack-reuse=none" } int main(int argc, char

[PATCH] Introduce tests for -fsanitize=use-after-scope

2016-05-06 Thread Martin Liška
Hi. This is a new test coverage for the new sanitizer option. Martin >From 753bfb3edb12c9f3fd13f320e308556f63330c97 Mon Sep 17 00:00:00 2001 From: marxin Date: Wed, 4 May 2016 12:57:05 +0200 Subject: [PATCH 2/2] Introduce tests for -fsanitize=use-after-scope

[SH][committed] Remove deprecated options

2016-05-06 Thread Oleg Endo
Hi, The attached patch removes some deprecated SH options. Tested on sh-elf with 'make all-gcc' and with 'make info dvi pdf'. Committed as r235960. Cheers, Oleg gcc/ChangeLog: * config/sh/sh.opt (madjust-unroll, minvalid-symbols, msoft-atomic, mspace): Remove deprecated

[PATCH, RFC] Introduce -fsanitize=use-after-scope

2016-05-06 Thread Martin Liška
Hello. I've started working on the patch couple of month go, basically after a brief discussion with Jakub on IRC. I'm sending the initial version which can successfully run instrumented tramp3d, postgresql server and Inkscape. It catches the basic set of examples which are added in following

[Bug sanitizer/70342] g++ -fsanitize=undefined never finishes compiling (>24h) in qtxmlpatterns test suite

2016-05-06 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70342 Marek Polacek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

Re: [PATCH] Fix memory leak in tree-inliner

2016-05-06 Thread Richard Biener
On Fri, May 6, 2016 at 12:10 PM, Martin Liška wrote: > Hi. > > I've spotted couple of occurrences of following memory leak seen by valgrind: > > malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) > operator new(unsigned long) (new_op.cc:50) >

[Bug sanitizer/70875] ICE in get_ubsan_type_info_for_type with -fsanitize=undefined

2016-05-06 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70875 Marek Polacek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug sanitizer/70875] ICE in get_ubsan_type_info_for_type with -fsanitize=undefined

2016-05-06 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70875 --- Comment #5 from Marek Polacek --- Author: mpolacek Date: Fri May 6 10:54:12 2016 New Revision: 235959 URL: https://gcc.gnu.org/viewcvs?rev=235959=gcc=rev Log: PR sanitizer/70875 * ubsan.c (get_ubsan_type_info_for_type):

Re: [PATCH] Fix coding style in tree-ssa-uninit.c

2016-05-06 Thread Richard Biener
On Fri, May 6, 2016 at 12:06 PM, Martin Liška wrote: > On 11/26/2015 10:04 PM, Bernd Schmidt wrote: >> As I said previously, the one to just replace whitespace is ok for now. >> Please ping the other one when stage1 opens (I expect it'll need changes by >> then). >> >> >> Bernd

[Bug sanitizer/70342] g++ -fsanitize=undefined never finishes compiling (>24h) in qtxmlpatterns test suite

2016-05-06 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70342 --- Comment #11 from Marek Polacek --- Author: mpolacek Date: Fri May 6 10:50:03 2016 New Revision: 235958 URL: https://gcc.gnu.org/viewcvs?rev=235958=gcc=rev Log: PR sanitizer/70342 * fold-const.c

Re: [PATCH PR70935, Regression 6,7]

2016-05-06 Thread Richard Biener
On Thu, May 5, 2016 at 5:19 PM, Yuri Rumyantsev wrote: > Hi All, > > Here is a simple patch which cures the problem with nonlegal > transformation of endless loop. THe fix is simply check that guard > edge destination is not loop latch block. > > Bootstrapping and regression

Re: [patch] Coalesce in more cases

2016-05-06 Thread Richard Biener
On Thu, May 5, 2016 at 5:08 PM, Eric Botcazou wrote: > Hi, > > gimple_can_coalesce_p is rather picky about the conditions under which SSA > names can be coalesced. In particular, when it comes to the type, it's: > > /* Now check the types. If the types are the same,

Re: Missing pointer dereference in tree-affine.c

2016-05-06 Thread Richard Biener
On Thu, May 5, 2016 at 4:19 PM, Richard Sandiford wrote: > wide_int_constant_multiple_p used: > > if (*mult_set && mult != 0) > return false; > > to check whether we had previously seen a nonzero multiple, but "mult" is > a pointer to the previous value

Re: [gimplefe] Regarding command line option handling

2016-05-06 Thread Richard Biener
On Wed, May 4, 2016 at 4:29 PM, Prasad Ghangal wrote: > On 4 May 2016 at 15:54, Richard Biener wrote: >> On Wed, May 4, 2016 at 11:46 AM, Prasad Ghangal >> wrote: >>> On 4 May 2016 at 13:02, Richard Biener

[Bug c++/70974] New: armv8-a+crc does not define __ARM_ACLE

2016-05-06 Thread noloader at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70974 Bug ID: 70974 Summary: armv8-a+crc does not define __ARM_ACLE Product: gcc Version: 4.9.2 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++

[SH][committed] Fix length of ic_invalidate_line_sh4a pattern

2016-05-06 Thread Oleg Endo
Hi, The attached patch fixes the length of the ic_invalidate_line_sh4a pattern. Tested on sh-elf with make -k check RUNTESTFLAGS="--target_board=sh-sim\{-m2/-ml,-m2/-mb, -m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" Committed as r235957. Cheers, Oleg gcc/ChangeLog: * config/sh/sh.md

Re: [PATCH 3/4] Extract deferred-location handling from jit

2016-05-06 Thread Richard Biener
On Wed, May 4, 2016 at 10:49 PM, David Malcolm wrote: > In order to faithfully load RTL dumps that contain references to > source locations, the RTL frontend needs to be able to parse file > and line information and turn then into location_t values. > > Unfortunately, the

Re: [PATCH 1/4] Make argv const char ** in read_md_files etc

2016-05-06 Thread Richard Biener
On Wed, May 4, 2016 at 10:49 PM, David Malcolm wrote: > This patch makes the argv param to read_md_files const, needed > so that the RTL frontend can call it on a const char *. > > While we're at it, it similarly makes const the argv for all > of the "main" functions of the

Re: [PATCH 2/4] Move name_to_pass_map into class pass_manager

2016-05-06 Thread Richard Biener
On Wed, May 4, 2016 at 10:49 PM, David Malcolm wrote: > The RTL frontend needs to be able to lookup passes by name. > > passes.c has global state name_to_pass_map (albeit static, scoped > to passes.c), for use by enable_disable_pass. > > Move it to be a field of class

Re: [PATCH] tail merge ICE

2016-05-06 Thread Richard Biener
On Wed, May 4, 2016 at 7:25 PM, Nathan Sidwell wrote: > This patch fixes an ICE Thomas observed in tree-ssa-tail-merge.c: > > On 05/03/16 06:34, Thomas Schwinge wrote: > >> I'm also seeing the following regression for C and C++, >> libgomp.oacc-c-c++-common/loop-auto-1.c

[SH][committed] Remove some workaround combine patterns

2016-05-06 Thread Oleg Endo
Hi, The attached patch removes some workaround combine patterns. As far as I remember this issue has been addressed by some match.pd patterns. In any case, CSiBE code size shows no difference and the SH specific testcases mentioned in the code pass without the patterns. Tested on sh-elf with

[Bug tree-optimization/70948] [7 Regression] r235622 caused gcc.c-torture/execute/va-arg-pack-1.c execution failure AArch64

2016-05-06 Thread jiwang at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70948 --- Comment #4 from Jiong Wang --- (In reply to Richard Biener from comment #3) > The following should fix it, I am going to test it on x86_64: > > Index: gcc/tree-ssa-structalias.c >

[SH][committed] Fix PR 58219

2016-05-06 Thread Oleg Endo
Hi, The attached patch fixes PR 58219. Tested on sh-elf with make -k check RUNTESTFLAGS="--target_board=sh-sim\{-m2/-ml,-m2/-mb, -m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" Committed as r235954. gcc/ChangeLog: PR target/58219 * config/sh/predicates.md

[Bug target/58219] [SH] mov.l insn length is wrong on SH2A

2016-05-06 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58219 --- Comment #2 from Oleg Endo --- Author: olegendo Date: Fri May 6 10:20:13 2016 New Revision: 235954 URL: https://gcc.gnu.org/viewcvs?rev=235954=gcc=rev Log: gcc/ PR target/58219 * config/sh/predicates.md

[Bug tree-optimization/70960] [7 Regression] ICE: tree check: expected ssa_name, have integer_cst in ifcvt_walk_pattern_tree, at tree-if-conv.c:2465

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70960 --- Comment #4 from Richard Biener --- Author: rguenth Date: Fri May 6 10:18:59 2016 New Revision: 235953 URL: https://gcc.gnu.org/viewcvs?rev=235953=gcc=rev Log: 2016-05-06 Richard Biener PR

[Bug tree-optimization/70960] [7 Regression] ICE: tree check: expected ssa_name, have integer_cst in ifcvt_walk_pattern_tree, at tree-if-conv.c:2465

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70960 Richard Biener changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[PATCH] Fix PR70960

2016-05-06 Thread Richard Biener
The following fixes $subject. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2016-05-06 Richard Biener PR tree-optimization/70960 * tree-if-conv.c (ifcvt_walk_pattern_tree): Handle non-SSA ops. *

[Bug tree-optimization/70948] [7 Regression] r235622 caused gcc.c-torture/execute/va-arg-pack-1.c execution failure AArch64

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70948 --- Comment #3 from Richard Biener --- The following should fix it, I am going to test it on x86_64: Index: gcc/tree-ssa-structalias.c === --- gcc/tree-ssa-structalias.c (revision

[PATCH] Fix memory leak in tree-inliner

2016-05-06 Thread Martin Liška
Hi. I've spotted couple of occurrences of following memory leak seen by valgrind: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so) operator new(unsigned long) (new_op.cc:50) remap_dependence_clique(copy_body_data*, unsigned short) (tree-inline.c:845)

[PATCH] Fix coding style in tree-ssa-uninit.c

2016-05-06 Thread Martin Liška
On 11/26/2015 10:04 PM, Bernd Schmidt wrote: > As I said previously, the one to just replace whitespace is ok for now. > Please ping the other one when stage1 opens (I expect it'll need changes by > then). > > > Bernd Hello. This part of the part remains to be installed from the previous

[Bug tree-optimization/70948] [7 Regression] r235622 caused gcc.c-torture/execute/va-arg-pack-1.c execution failure AArch64

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70948 --- Comment #2 from Richard Biener --- I don't have an arm execution environment so maybe you can help me reducing/analyzing this a bit. The two places the optimization happens on are during PRE in foo1: Visiting control stmt ending BB 34: if

[Bug target/70973] x86: Can the __atomic_*() operations be made to list the LOCK prefixes?

2016-05-06 Thread dhowells at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70973 --- Comment #1 from dhowells at redhat dot com --- There may be space that can be used in the memorder parameter: "The memory order parameter is a signed int, but only the lower 16 bits are reserved for the memory order. The remainder of the

[gomp4.5] Allow more than 64 clauses in gfc_match_omp_clauses

2016-05-06 Thread Jakub Jelinek
Hi! With 32 OpenMP clauses and 27 further OpenACC ones, I can't add 10 further clauses I need for OpenMP. So, this patch implements using C++ classes a framework where the code can use mostly what it used, yet it is now up to 128 bits, while being not really more expensive at -O2 than passing

[Bug target/70973] New: x86: Can the __atomic_*() operations be made to list the LOCK prefixes?

2016-05-06 Thread dhowells at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70973 Bug ID: 70973 Summary: x86: Can the __atomic_*() operations be made to list the LOCK prefixes? Product: gcc Version: 6.0 Status: UNCONFIRMED Severity: normal

[SH][committed] Add some more missing div0s cases

2016-05-06 Thread Oleg Endo
Hi, The attached patch addresses some of the remaining things as mentioned in the PR. Tested on sh-elf with make -k check RUNTESTFLAGS="--target_board=sh-sim\{-m2/-ml,-m2/-mb, -m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" Committed as r235952. Cheers, Oleg gcc/ChangeLog: PR

[Bug target/52933] SH Target: Use div0s for integer sign comparisons

2016-05-06 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52933 --- Comment #8 from Oleg Endo --- Author: olegendo Date: Fri May 6 09:52:35 2016 New Revision: 235952 URL: https://gcc.gnu.org/viewcvs?rev=235952=gcc=rev Log: gcc/ PR target/52933 * config/sh/sh.md (*cmp_div0s_7, *cmp_div0s_8):

Re: Fix for PR68159 in Libiberty Demangler (6)

2016-05-06 Thread Jakub Jelinek
On Fri, May 06, 2016 at 05:01:14PM +0800, Marcel Böhme wrote: > The patch that is attached now is bootstrapped and regression tested on > x86_64-pc-linux-gnu. > > > > > This file is used not just in the various tools like binutils or gdb, but > > also in libstdc++, where it used e.g. in the

[Bug sanitizer/70875] ICE in get_ubsan_type_info_for_type with -fsanitize=undefined

2016-05-06 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70875 --- Comment #4 from Marek Polacek --- Author: mpolacek Date: Fri May 6 09:47:25 2016 New Revision: 235951 URL: https://gcc.gnu.org/viewcvs?rev=235951=gcc=rev Log: PR sanitizer/70875 * ubsan.c (get_ubsan_type_info_for_type):

[SH][committed] Add another rotcr variant

2016-05-06 Thread Oleg Endo
Hi, The attached patch adds another combine pattern variant for the SH rotcr instruction. Tested on sh-elf with make -k check RUNTESTFLAGS="--target_board=sh-sim\{-m2/-ml,-m2/-mb, -m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" Committed as r235950. Cheers, Oleg gcc/ChangeLog: PR

Re: [PATCH v2] Allocate constant size dynamic stack space in the prologue

2016-05-06 Thread Dominik Vogt
> diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c > index 21f21c9..4d48afd 100644 > --- a/gcc/cfgexpand.c > +++ b/gcc/cfgexpand.c ... > @@ -1099,8 +1101,10 @@ expand_stack_vars (bool (*pred) (size_t), struct > stack_vars_data *data) > >/* If there were any, allocate space. */ >

Re: [PATCH GCC]Proving no-trappness for array ref in tree if-conv using loop niter information.

2016-05-06 Thread Bin.Cheng
On Fri, May 6, 2016 at 10:40 AM, Bin.Cheng wrote: > On Tue, May 3, 2016 at 11:08 AM, Richard Biener > wrote: >> On Tue, May 3, 2016 at 12:01 PM, Bin.Cheng wrote: >>> On Mon, May 2, 2016 at 10:00 AM, Richard Biener >>>

[Bug target/54089] [SH] Refactor shift patterns

2016-05-06 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54089 --- Comment #45 from Oleg Endo --- Author: olegendo Date: Fri May 6 09:41:57 2016 New Revision: 235950 URL: https://gcc.gnu.org/viewcvs?rev=235950=gcc=rev Log: gcc/ PR target/54089 * config/sh/sh.md (*rotcr): Add another

Re: [PATCH GCC]Proving no-trappness for array ref in tree if-conv using loop niter information.

2016-05-06 Thread Bin.Cheng
On Tue, May 3, 2016 at 11:08 AM, Richard Biener wrote: > On Tue, May 3, 2016 at 12:01 PM, Bin.Cheng wrote: >> On Mon, May 2, 2016 at 10:00 AM, Richard Biener >> wrote: >>> On Fri, Apr 29, 2016 at 5:05 PM, Bin.Cheng

Re: [PATCH v2] Allocate constant size dynamic stack space in the prologue

2016-05-06 Thread Dominik Vogt
Updated version of the patch described below. Apart from fixing a bug and adding a test, the new logic is now used always, for all targets. The discussion of the original patch starts here: https://gcc.gnu.org/ml/gcc-patches/2015-11/msg03052.html The new patch has been bootstrapped and

Re: [ubsan PATCH] Fix ICE with bounds checking on VLA-in-a-struct (PR sanitizer/70875)

2016-05-06 Thread Marek Polacek
On Fri, May 06, 2016 at 11:29:33AM +0200, Jakub Jelinek wrote: > On Fri, May 06, 2016 at 11:22:41AM +0200, Marek Polacek wrote: > > A program containing an array of structs containing a VLA caused ICE with > > UBSAN > > bounds checking, because in get_ubsan_type_info_for_type we asserted that >

[Bug sanitizer/69443] -fsanitize=address should add -lpthread to linker call

2016-05-06 Thread y.gribov at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69443 Yury Gribov changed: What|Removed |Added CC||y.gribov at samsung dot com --- Comment

[Bug fortran/70937] [7 Regression] ICE: tree code ‘ssa_name’ is not supported in LTO streams

2016-05-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70937 Richard Biener changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment

Re: [ubsan PATCH] Fix ICE with bounds checking on VLA-in-a-struct (PR sanitizer/70875)

2016-05-06 Thread Jakub Jelinek
On Fri, May 06, 2016 at 11:22:41AM +0200, Marek Polacek wrote: > A program containing an array of structs containing a VLA caused ICE with > UBSAN > bounds checking, because in get_ubsan_type_info_for_type we asserted that the > size of a type fits uhwi, which implies it is an INTEGER_CST. But

[ubsan PATCH] Fix ICE with bounds checking on VLA-in-a-struct (PR sanitizer/70875)

2016-05-06 Thread Marek Polacek
A program containing an array of structs containing a VLA caused ICE with UBSAN bounds checking, because in get_ubsan_type_info_for_type we asserted that the size of a type fits uhwi, which implies it is an INTEGER_CST. But that's not the case for a struct with VLA. However, the assert here is

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-06 Thread Martin Liška
Hi. Honza asked me to explain the change more verbosely. The patch simplify enhances verbose dump of IVOPTS so that # of iterations is printed. Apart from that it also prints invariant expression that are used during the algorithm which considers a set of candidates which is improved. Main

[Bug target/70904] ICE: Max. number of generated reload insns per insn is achieved (90) with -fno-split-wide-types @ aarch64

2016-05-06 Thread jiwang at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70904 Jiong Wang changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug ada/70969] [7 regression] bootstrap failure: s-unstyp.ads:40:15: violation of No_Elaboration_Code_All at line 42

2016-05-06 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70969 Eric Botcazou changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ada/70969] [7 regression] bootstrap failure: s-unstyp.ads:40:15: violation of No_Elaboration_Code_All at line 42

2016-05-06 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70969 --- Comment #6 from Eric Botcazou --- Author: ebotcazou Date: Fri May 6 09:12:09 2016 New Revision: 235948 URL: https://gcc.gnu.org/viewcvs?rev=235948=gcc=rev Log: PR ada/70969 * system-darwin-ppc64.ads: Add pragma

[Bug ada/70969] [7 regression] bootstrap failure: s-unstyp.ads:40:15: violation of No_Elaboration_Code_All at line 42

2016-05-06 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70969 --- Comment #4 from Eric Botcazou --- Author: ebotcazou Date: Fri May 6 09:11:38 2016 New Revision: 235946 URL: https://gcc.gnu.org/viewcvs?rev=235946=gcc=rev Log: PR ada/70969 * system-darwin-ppc64.ads: Add pragma

<    1   2   3   >