Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-26 Thread Maxim Ostapenko
Thank you all for your help! Done in r215633. -Maxim On 09/25/2014 11:05 PM, Jeff Law wrote: On 09/23/14 01:14, Maxim Ostapenko wrote: 2014-09-04 Jakub Jelinek Max Ostapenko * common.opt: New option. * doc/invoke.texi: Describe new option. * gcc.c (execute): Don't free

Re: [debug-early] fix fortran regressions

2014-09-26 Thread Richard Biener
On Thu, Sep 25, 2014 at 8:11 PM, Aldy Hernandez wrote: > push_cfun() fails when there's no cfun stack. With this patch, we use > set_cfun if not stack is available. > > This fixes the 16 Fortran guality regressions. > > Now guality tests all pass, for all languages. > > Committed to branch. Hmm,

Re: Fix TYPE_MAIN_VARIANT made by ipa-prop

2014-09-26 Thread Richard Biener
On Fri, Sep 26, 2014 at 12:14 AM, Jan Hubicka wrote: > Hi, > according to my type checker, ipa-prop is only place where we produce a > variant of > a METHOD_TYPE that is FUNCTION_TYPE or viarant that has different parameters. > > The code in question is producing new prototype to remove unused ar

Re: [shrink-wrap] should not sink instructions which may cause trap ?

2014-09-26 Thread Richard Biener
On Fri, Sep 26, 2014 at 12:49 AM, Jiong Wang wrote: > 2014-09-25 14:07 GMT+01:00 Jiong Wang : >> >> On 25/09/14 12:25, Christophe Lyon wrote: >>> >>> I have observed regressions in the g++ testsuite: pr49847 now FAILs >>> after this patch. >> >> no. >> >> even without my patch, the regression

[PATCH] Support for BIT_FIELD_REF in asan.c

2014-09-26 Thread Marat Zakirov
Hi all! Here's a patch which instruments byte-aligned BIT_FIELD_REFs. During GCC asan-bootstrap and Linux kernel build I didn't find any cases where BIT_FIELD_REF is not 8 bits aligned. But I do not have sufficient confidence to replace current return if BIT_FIELD_REF is misaligned to assert.

Re: [Patch 2/4] Hack out a use of MOVE_RATIO in tree-inline.c

2014-09-26 Thread Richard Biener
On Thu, Sep 25, 2014 at 4:57 PM, James Greenhalgh wrote: > > Hi, > > This patch hookizes the use of MOVE_RATIO in > tree-inline.c:estimate_move_cost as TARGET_ESTIMATE_BLOCK_COPY_NINSNS. > This hook should return an estimate for the number of instructions > which will be emitted to copy a block of

[PATCH i386 AVX512] [57/n] Extend blend/cmp/brodcast insn patterns.

2014-09-26 Thread Kirill Yukhin
Hello, Patch in the bottom extends blend/cmp/brodcast insn patterns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_insn "avx512f_blendm"): Delete. (define_insn "_blendm"): New. (def

Re: [Patchv2 3/4] Control SRA and IPA-SRA by a param rather than MOVE_RATIO

2014-09-26 Thread Richard Biener
On Thu, Sep 25, 2014 at 4:57 PM, James Greenhalgh wrote: > > Hi, > > After hookizing MOVE_BY_PIECES_P and migrating tree-inline.c, we are > left with only one user of MOVE_RATIO - deciding the maximum size of > aggregate for SRA. > > Past discussions have made it clear [1] that keeping this use of

Re: [Patch 1/4] Hookize MOVE_BY_PIECES_P, remove most uses of MOVE_RATIO

2014-09-26 Thread Richard Biener
On Thu, Sep 25, 2014 at 5:08 PM, Steven Bosscher wrote: > On Thu, Sep 25, 2014 at 4:57 PM, James Greenhalgh wrote: >> * doc/tm.texi.in (MOVE_BY_PIECES_P): Reduce documentation to a stub >> describing that this macro is deprecated. > > Remove it entirely and poison it in system.h? >

[PATCH][match-and-simplify] Merge some more code-gen code

2014-09-26 Thread Richard Biener
Bootstrapped on x86_64-unknown-linux-gnu, applied. Richard. 2014-09-26 Richard Biener * genmatch.c (struct dt_node): Merge gen_gimple and gen_generic into gen, merge gen_gimple_kids and gen_generic_kids into gen_kids. (struct dt_operand): Likewise. Index: gcc

Re: [Bug libstdc++/62313] Data race in debug iterators

2014-09-26 Thread Jonathan Wakely
On 26/09/14 00:00 +0200, François Dumont wrote: Apart from those minor adjustments I think this looks good, but I'd like to know that it has been tested with -fsanitize=thread, even if only lightly tested. Hi Dmitry, who reported the bug, confirmed the fix. Can I go ahead and commit ?

Re: [PATCH, i386, Pointer Bounds Checker 33/x] MPX ABI

2014-09-26 Thread Ilya Enkovich
Adding Vladimir. Ilya 2014-09-25 13:46 GMT+04:00 Ilya Enkovich : > 2014-09-25 1:51 GMT+04:00 Ilya Enkovich : >> 2014-09-24 23:09 GMT+04:00 Jeff Law : >>> On 09/24/14 07:13, Ilya Enkovich wrote: I tried to generate PARALLEL with all regs set by call. Here is a memset call I got: >>

Re: [PATCH, Pointer Bounds Checker 14/x] Pointer Bounds Checker passes

2014-09-26 Thread Ilya Enkovich
Ping 2014-06-06 10:54 GMT+04:00 Ilya Enkovich : > On 03 Jun 10:59, Richard Biener wrote: >> On Mon, Jun 2, 2014 at 5:13 PM, Ilya Enkovich wrote: >> > 2014-06-02 17:37 GMT+04:00 Richard Biener : >> >> On Mon, Jun 2, 2014 at 2:44 PM, Ilya Enkovich >> >> wrote: >> >>> 2014-06-02 15:35 GMT+04:00 Ri

Re: [PATCH 2/5] Existing call graph infrastructure enhancement

2014-09-26 Thread Martin Liška
On 09/24/2014 05:01 PM, Jan Hubicka wrote: Hi. Following patch enhances API functions to be ready for main patch of this patchset. Ready for thunk? Thank you, Martin gcc/ChangeLog: 2014-09-21 Martin Liška * cgraph.c (cgraph_node::release_body): New argument keep_arguments

[PATCH i386 AVX512] [58/n] Add vpmul[u]dq insn patterns.

2014-09-26 Thread Kirill Yukhin
Hello, Patch in the bottom adds support for vpmul[u]dq insn patterns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_expand "vec_widen_umult_even_v8si"): Add masking. (define_insn "*vec_wide

[PATCH i386 AVX512] [59/n] Add vptest[n]m, ucmp, cmpeq insn patterns.

2014-09-26 Thread Kirill Yukhin
Hello, Patch in the bottom adds support for vptest[n]m, ucmp, cmpeq. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c (ix86_expand_args_builtin): Handle CODE_FOR_avx512vl_cmpv4di3_mask, CODE_FOR_avx51

[PATCH][match-and-simplify] Enable conversions properly for GENERIC

2014-09-26 Thread Richard Biener
This fixes a ??? and handles NOP_EXPR and CONVERT_EXPR matching similar to GIMPLE by using CASE_CONVERT. This uncovers a mismatch between fold-const.c and tree-ssa-forwprop.c transforms which try to do opposite things with ((T) X) & CST vs. (T) (X & CST). I have disabled the forwprop transform o

[PATCH i386 AVX512] [60/n] Update 128bit ashrv insn pattern.

2014-09-26 Thread Kirill Yukhin
Hello, This tiny patch extends 128bit ashrv expander. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_mode_iterator VI128_128 [V16QI V8HI V2DI]): Delete. (define_expand "vashr3"): Add masking

Re: ptx preliminary address space fixes [1/4]

2014-09-26 Thread Bernd Schmidt
On 09/16/2014 02:59 PM, Richard Biener wrote: On Tue, Sep 16, 2014 at 1:24 PM, Bernd Schmidt wrote: Ok, so testing seems to show that nothing breaks with the ARRAY_TYPE special case removed. However, I remembered another reason to do this, and it's for consistency with how address spaces are re

Re: ptx preliminary address space fixes [1/4]

2014-09-26 Thread Richard Biener
On Fri, Sep 26, 2014 at 2:00 PM, Bernd Schmidt wrote: > On 09/16/2014 02:59 PM, Richard Biener wrote: >> >> On Tue, Sep 16, 2014 at 1:24 PM, Bernd Schmidt >> wrote: >>> >>> Ok, so testing seems to show that nothing breaks with the ARRAY_TYPE >>> special >>> case removed. However, I remembered ano

Re: ptx preliminary address space fixes [1/4]

2014-09-26 Thread Bernd Schmidt
On 09/26/2014 02:05 PM, Richard Biener wrote: If currently address-space support matches up with the C frontend and the C standard then the middle-end has to cope with that. In this case, cope with array element types not having address-space qualifiers. That's the opposite of what happens. The

[PATCH] Fix PR preprocessor/58893 access to uninitialized memory

2014-09-26 Thread Bernd Edlinger
Hi, this patch fixes PR58893, which is an access to uninitialized memory, which may or may not crash in linemap_resolve_location, or just print error messages with bogus location. When the first -include file is processed we have the case, where pfile->cur_token == pfile->cur_run->base, this is

Re: [PATCH] Fix PR preprocessor/58893 access to uninitialized memory

2014-09-26 Thread Marek Polacek
On Fri, Sep 26, 2014 at 02:16:05PM +0200, Bernd Edlinger wrote: > Boot-Strapped and Regression-tested on x86_64-linux-gnu > Ok for trunk? -ENOPATCH. Marek

Re: [PATCH 3/5] IPA ICF pass

2014-09-26 Thread Martin Liška
On 07/17/2014 05:05 PM, Martin Liška wrote: On 07/06/2014 12:53 AM, Jan Hubicka wrote: On Fri, 20 Jun 2014, Trevor Saunders wrote: +@item -fipa-icf +@opindex fipa-icf +Perform Identical Code Folding for functions and read-only variables. I would perhaps explicitly say that the optimizations r

FW: [PATCH] Fix PR preprocessor/58893 access to uninitialized memory

2014-09-26 Thread Bernd Edlinger
Aehm, sorry., again, with patch files. > > Hi, > > this patch fixes PR58893, which is an access to uninitialized memory, which > may or may not crash in > linemap_resolve_location, or just print error messages with bogus location. > > When the first -include file is processed we have the case,

Re: [PATCH 4/5] Existing tests fix

2014-09-26 Thread Martin Liška
On 06/30/2014 02:11 PM, Martin Liška wrote: On 06/17/2014 09:52 PM, Jeff Law wrote: On 06/13/14 04:48, mliska wrote: Hi, many tests rely on a precise number of scanned functions in a dump file. If IPA ICF decides to merge some function and(or) read-only variables, counts do not match. Ma

Re: ptx preliminary address space fixes [1/4]

2014-09-26 Thread Richard Biener
On Fri, Sep 26, 2014 at 2:14 PM, Bernd Schmidt wrote: > On 09/26/2014 02:05 PM, Richard Biener wrote: >> >> If currently address-space support matches up with the C frontend >> and the C standard then the middle-end has to cope with that. >> In this case, cope with array element types not having a

Re: ptx preliminary address space fixes [1/4]

2014-09-26 Thread Bernd Schmidt
On 09/26/2014 02:26 PM, Richard Biener wrote: On Fri, Sep 26, 2014 at 2:14 PM, Bernd Schmidt wrote: On 09/26/2014 02:05 PM, Richard Biener wrote: If currently address-space support matches up with the C frontend and the C standard then the middle-end has to cope with that. In this case, cope

[PATCH i386 AVX512] [61/n] Update FP logic insn patterns.

2014-09-26 Thread Kirill Yukhin
Hello, This patch extends andnot and any_logic insn patterns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_insn "_andnot3"): Add masking, use VF_128_256 mode iterator and update assembler

Re: [PATCH 1/n] OpenMP 4.0 offloading infrastructure

2014-09-26 Thread Ilya Verbin
Hi, The patch has been updated: On 17 Sep 15:45, Jakub Jelinek wrote: > Looks mostly ok, just some nits. But see the patch I've just posted, > perhaps we want to tweak the --enable-offload-targets arguments. And Now the targets in --enable-offload-targets can have optional path to the build or

Re: ptx preliminary address space fixes [1/4]

2014-09-26 Thread Richard Biener
On Fri, Sep 26, 2014 at 2:28 PM, Bernd Schmidt wrote: > On 09/26/2014 02:26 PM, Richard Biener wrote: >> >> On Fri, Sep 26, 2014 at 2:14 PM, Bernd Schmidt >> wrote: >>> >>> On 09/26/2014 02:05 PM, Richard Biener wrote: If currently address-space support matches up with the C fronte

Re: [GOMP4, RFC] OpenMP4 offload support for Intel PHI targets.

2014-09-26 Thread Ilya Verbin
Hi, I also rebased and updated our branch: https://gcc.gnu.org/git/?p=gcc.git;a=shortlog;h=refs/heads/kyukhin/gomp4-offload It contains fixes for the issues, mentioned in "Offloading not relocatable". https://gcc.gnu.org/wiki/Offloading was updated accordingly. -- Ilya

Re: [PATCH 2/4] [AARCH64,NEON] Convert arm_neon.h to use new builtins for vld[234](q?)_lane_*

2014-09-26 Thread Tejas Belagod
On 26/09/14 02:16, Charles Baylis wrote: On 19 September 2014 12:21, Tejas Belagod wrote: The reason we avoided using type-punning using unions was that reload would get confused with potential subreg(mem) that could be introduced because of memory xfer caused by unions and large int modes. As

[PATCH] Avoid an unused stack frame for -mprofile-kernel profiling on leaf functions.

2014-09-26 Thread Anton Blanchard
gcc/: 2014-09-25 Anton Blanchard PR target/63354 * config/rs6000/rs6000.c (rs6000_keep_leaf_when_profiled): New function. * config/rs6000/linux64.h (TARGET_KEEP_LEAF_WHEN_PROFILED): Define. --- gcc/config/rs6000/linux64.h | 3 +++ gcc/config/rs6000/rs6000.c | 9 +

Re: Avoid privatization of TLS variables

2014-09-26 Thread Alan Modra
On Fri, Sep 26, 2014 at 04:17:14AM +0200, Jan Hubicka wrote: > I was building libreoffice with profile feedback and I run into a message > > cannot load any more object with static TLS > > that took me a while to track as I did not see where static TLS is comming > out. > Ian pointed out to me t

Re: [PATCH] Put all MAINTAINERS email addresses into <...>

2014-09-26 Thread Segher Boessenkool
On Thu, Sep 25, 2014 at 11:10:00PM +0200, Jan-Benedict Glaw wrote: > Resending this email. Seems some spam filter ate it due to the many > email addresses... Will now all/most/many further patches to MAINTAINERS hit spam filters as well? Segher

Re: [Patch, AArch64] Enable Address sanitizer and UB sanitizer

2014-09-26 Thread Christophe Lyon
On 21 September 2014 20:07, Christophe Lyon wrote: > On 17 September 2014 12:48, Marcus Shawcroft > wrote: >> On 9 September 2014 13:08, Christophe Lyon >> wrote: >>> On 9 September 2014 12:03, wrote: > On Sep 9, 2014, at 2:50 AM, Marcus Shawcroft > wrote: > > +st

Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-26 Thread Rainer Orth
Hi Maxim, > Thank you all for your help! > > Done in r215633. unfortuntely, the applied patch cannot have been tested properly and breaks native i386-pc-solaris2.11 (and every other) bootstrap: /vol/gcc/src/hg/trunk/local/gcc/gcc.c: In function 'attempt_status run_attempt(const char**, const ch

Re: [PATCH] Put all MAINTAINERS email addresses into <...>

2014-09-26 Thread Steven Bosscher
On Fri, Sep 26, 2014 at 3:09 PM, Segher Boessenkool wrote: > On Thu, Sep 25, 2014 at 11:10:00PM +0200, Jan-Benedict Glaw wrote: >> Resending this email. Seems some spam filter ate it due to the many >> email addresses... > > Will now all/most/many further patches to MAINTAINERS hit spam filters > a

Re: [PATCH] Put all MAINTAINERS email addresses into <...>

2014-09-26 Thread Trevor Saunders
On Fri, Sep 26, 2014 at 03:14:16PM +0200, Steven Bosscher wrote: > On Fri, Sep 26, 2014 at 3:09 PM, Segher Boessenkool wrote: > > On Thu, Sep 25, 2014 at 11:10:00PM +0200, Jan-Benedict Glaw wrote: > >> Resending this email. Seems some spam filter ate it due to the many > >> email addresses... > > >

Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-26 Thread Thomas Schwinge
Hi! On Fri, 26 Sep 2014 12:04:45 +0400, Maxim Ostapenko wrote: > Thank you all for your help! > > Done in r215633. > > -Maxim > On 09/25/2014 11:05 PM, Jeff Law wrote: > > On 09/23/14 01:14, Maxim Ostapenko wrote: > >> > >> > >> 2014-09-04 Jakub Jelinek > >> Max Ostapenko > >> > >>

Re: [Patch AArch64 4/4] Wire up New target hooks

2014-09-26 Thread James Greenhalgh
On Thu, Sep 25, 2014 at 03:57:36PM +0100, James Greenhalgh wrote: > > Hi, > > This patch wires up our new target hooks for AArch64. This also means > we can bring back the two failing SRA tests. > > Bootstrapped on AArch64 with no issues. > > OK for trunk? No way! This patch is nonsense as it

Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-26 Thread Maxim Ostapenko
Ugh, sorry. Thanks for fixing this. On 09/26/2014 05:23 PM, Thomas Schwinge wrote: Hi! On Fri, 26 Sep 2014 12:04:45 +0400, Maxim Ostapenko wrote: Thank you all for your help! Done in r215633. -Maxim On 09/25/2014 11:05 PM, Jeff Law wrote: On 09/23/14 01:14, Maxim Ostapenko wrote: 2014-0

Re: [PATCH IRA] update_equiv_regs fails to set EQUIV reg-note for pseudo with more than one definition

2014-09-26 Thread Felix Yang
Hi Jeff, Thanks for the suggestions. I updated the patch accordingly. 1. Both my employer(Huawei) and I have signed the copyright assignments with FSF. These assignments are already sent via post two days ago and hopefully should reach FSF in one week. Maybe it's OK to com

[PATCH i386 AVX512] [62/n] Add vpmaddubsw,vdbpsadbw insn patterns.

2014-09-26 Thread Kirill Yukhin
Hello, This patch introduces patterns for vpmaddubsw and vdbpsadbw insn. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_c_enum "unspec"): Add UNSPEC_DBPSADBW, UNSPEC_PMADDUBSW512. (define_in

Re: [PATCH C++] - SD-6 Implementation Part 2 - __has_include macro and C++ language feature macros.

2014-09-26 Thread Ed Smith-Rowland
On 09/25/2014 01:40 PM, Jason Merrill wrote: On 09/01/2014 09:41 PM, Ed Smith-Rowland wrote: + cpp_define (pfile, "__cpp_attribute_deprecated=201309"); Don't we support attribute deprecated in C++11? Jason We support [[gnu::deprecated]] in C++11 bit not [[deprecated]] until C++14. Ed

RE: Fix for "FAIL: tmpdir-gcc.dg-struct-layout-1/t028 c_compat_x_tst.o compile, (internal compiler error)"

2014-09-26 Thread David Sherwood
Hi Vladimir, Sorry this took so long. I have tidied up the patch as you suggested and fixed some style issues. Hope this looks better now. Thanks! David. 2014-09-26 David Sherwood * ira-int.h (ira_allocno): Add "wmode" field. * ira-build.c (create_insn_allocnos): Add new "pa

Re: [RFC][ARM]: Fix reload spill failure (PR 60617)

2014-09-26 Thread Christophe Lyon
Ramana, On 7 July 2014 13:48, Venkataramanan Kumar wrote: > Hi Ramana/Maxim, > > > On 18 June 2014 16:05, Venkataramanan Kumar > wrote: >> Hi Ramana, >> >> On 18 June 2014 15:29, Ramana Radhakrishnan >> wrote: >>> On Mon, Jun 16, 2014 at 1:53 PM, Venkataramanan Kumar >>> wrote: Hi Mainta

[jit] Eliminate fixed-size buffer for a context's first error message

2014-09-26 Thread David Malcolm
On Wed, 2014-09-24 at 22:04 -0600, Jeff Law wrote: On 09/24/14 14:24, Joseph S. Myers wrote: > > On Wed, 24 Sep 2014, David Malcolm wrote: > > > >> The ideal I'm aiming for here is that a well-behaved library should > >> never abort, so I've rewritten these functions to use vasprintf, and > >> adde

Re: [PATCH 3/5] IPA ICF pass

2014-09-26 Thread Markus Trippelsdorf
On 2014.09.26 at 14:20 +0200, Martin Liška wrote: > After couple of weeks I spent with fixing new issues connected to the > pass: 1) Inliner failed in case I created a thunk and release body of > a function. In such situation we need to preserve DECL_ARGUMENTS. I > added new argument for: cgraph_no

Re: [shrink-wrap] should not sink instructions which may cause trap ?

2014-09-26 Thread Jiong Wang
On 26/09/14 09:36, Richard Biener wrote: On Fri, Sep 26, 2014 at 12:49 AM, Jiong Wang wrote: 2014-09-25 14:07 GMT+01:00 Jiong Wang : On 25/09/14 12:25, Christophe Lyon wrote: I have observed regressions in the g++ testsuite: pr49847 now FAILs after this patch. no. even without my patch, th

Re: [shrink-wrap] should not sink instructions which may cause trap ?

2014-09-26 Thread Jiong Wang
On 26/09/14 15:45, Jiong Wang wrote: On 26/09/14 09:36, Richard Biener wrote: On Fri, Sep 26, 2014 at 12:49 AM, Jiong Wang wrote: 2014-09-25 14:07 GMT+01:00 Jiong Wang : On 25/09/14 12:25, Christophe Lyon wrote: I have observed regressions in the g++ testsuite: pr49847 now FAILs after this p

Re: [COMMITTED][PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-09-26 Thread H.J. Lu
On Thu, Sep 25, 2014 at 9:43 AM, Jiong Wang wrote: > > On 25/09/14 17:24, Jeff Law wrote: >> >> On 09/25/14 09:04, Jiong Wang wrote: >>> >>> new patch updated. >>> >>> pass bootstrap and no regression, both check-gcc and check-g++, on the >>> x86. >>> >>> OK for trunk? >>> >>> thanks. >>> >>> gcc/

Re: [COMMITTED][PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-09-26 Thread Jiong Wang
On 26/09/14 16:05, H.J. Lu wrote: On Thu, Sep 25, 2014 at 9:43 AM, Jiong Wang wrote: On 25/09/14 17:24, Jeff Law wrote: On 09/25/14 09:04, Jiong Wang wrote: new patch updated. pass bootstrap and no regression, both check-gcc and check-g++, on the x86. OK for trunk? thanks. gcc/ * s

Re: ptx preliminary address space fixes [1/4]

2014-09-26 Thread Bernd Schmidt
On 09/26/2014 02:42 PM, Richard Biener wrote: On Fri, Sep 26, 2014 at 2:28 PM, Bernd Schmidt wrote: On 09/26/2014 02:26 PM, Richard Biener wrote: On Fri, Sep 26, 2014 at 2:14 PM, Bernd Schmidt wrote: On 09/26/2014 02:05 PM, Richard Biener wrote: If currently address-space support matche

[jit] Add a test of using very long names

2014-09-26 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/testsuite/ChangeLog.jit: * jit.dg/test-long-names.c: New test case. * jit.dg/all-non-failing-tests.h: Add test-long-names.c * jit.dg/test-combination.c (create_code): Likewise. (verify_code): Likewise. * jit.dg/test-thre

Re: [PATCH][match-and-simplify] Enable conversions properly for GENERIC

2014-09-26 Thread Jason Merrill
On 09/26/2014 06:54 AM, Richard Biener wrote: It also uncovers that the C++ FE uses a mix of NOP_EXPR and CONVERT_EXPR both when building expressions and when checking for them. Jason - is there any difference between NOP_EXPR and CONVERT_EXPR as far as the C++ FE is concerned? There are a few

Re: [COMMITTED][PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-09-26 Thread H.J. Lu
On Fri, Sep 26, 2014 at 8:14 AM, Jiong Wang wrote: > > On 26/09/14 16:05, H.J. Lu wrote: >> >> On Thu, Sep 25, 2014 at 9:43 AM, Jiong Wang wrote: >>> >>> On 25/09/14 17:24, Jeff Law wrote: On 09/25/14 09:04, Jiong Wang wrote: > > new patch updated. > > pass bootstrap and

Re: [COMMITTED][PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-09-26 Thread Jiong Wang
On 26/09/14 16:28, H.J. Lu wrote: On Fri, Sep 26, 2014 at 8:14 AM, Jiong Wang wrote: could you mark that testcase using something like "dg-require-effective-target lp64"? I checked in this patch to skip it on ia32. great, thanks ! -- Jiong

Re: [PATCH] fix hardreg_cprop to honor HARD_REGNO_MODE_OK.

2014-09-26 Thread Ilya Tocar
On 25 Sep 13:14, Jeff Law wrote: > On 09/01/14 04:29, Ilya Tocar wrote: > >>> > >>>AVX512 added new 16 xmm registers (xmm16-xmm31). > >>>Those registers require evex encoding. > >>>Only 512-bit wide versions of instructions have evex encoding with > >>>avx512f, but all versions have it with avx512v

Re: [PATCH] Fix PR63266: Keep track of impact of sign extension in bswap

2014-09-26 Thread Christophe Lyon
On 26 September 2014 04:25, Thomas Preud'homme wrote: >> From: Christophe Lyon [mailto:christophe.l...@linaro.org] >> Sent: Thursday, September 25, 2014 10:08 PM >> > >> While attempting to try this, I noticed that more precisely the test >> is currently UNSUPPORTED on aarch64_be, >> which is beca

Re: [shrink-wrap] should not sink instructions which may cause trap ?

2014-09-26 Thread Jeff Law
On 09/26/14 08:50, Jiong Wang wrote: if (may_trap_p (x)) don't sink this instruction. any comments? Should be checking if x may throw internally instead. Richard, thanks for the suggestion, have used insn_could_throw_p to do the check, which will only do the check when flag_ex

Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-26 Thread Jeff Law
On 09/26/14 07:23, Thomas Schwinge wrote: Hi! On Fri, 26 Sep 2014 12:04:45 +0400, Maxim Ostapenko wrote: Thank you all for your help! Done in r215633. -Maxim On 09/25/2014 11:05 PM, Jeff Law wrote: On 09/23/14 01:14, Maxim Ostapenko wrote: 2014-09-04 Jakub Jelinek Max Ostapenk

Re: [PATCH] Avoid an unused stack frame for -mprofile-kernel profiling on leaf functions.

2014-09-26 Thread David Edelsohn
On Fri, Sep 26, 2014 at 9:02 AM, Anton Blanchard wrote: > > gcc/: > > 2014-09-25 Anton Blanchard > > PR target/63354 > * config/rs6000/rs6000.c (rs6000_keep_leaf_when_profiled): New > function. > * config/rs6000/linux64.h (TARGET_KEEP_LEAF_WHEN_PROFILED): Define. Okay.

Re: ptx preliminary address space fixes [1/4]

2014-09-26 Thread Richard Biener
On September 26, 2014 5:14:24 PM CEST, Bernd Schmidt wrote: >On 09/26/2014 02:42 PM, Richard Biener wrote: >> On Fri, Sep 26, 2014 at 2:28 PM, Bernd Schmidt > wrote: >>> On 09/26/2014 02:26 PM, Richard Biener wrote: On Fri, Sep 26, 2014 at 2:14 PM, Bernd Schmidt > wrote: >

Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-26 Thread Thomas Schwinge
Hi! On Fri, 26 Sep 2014 10:18:32 -0600, Jeff Law wrote: > On 09/26/14 07:23, Thomas Schwinge wrote: > > On Fri, 26 Sep 2014 12:04:45 +0400, Maxim Ostapenko > > wrote: > >> Done in r215633. > > This is causing compiler warnings, respectively bootstrap errors: [...] > > OK to fix as follows? O

Re: [PATCH 2/2] Add patch for debugging compiler ICEs.

2014-09-26 Thread Jeff Law
On 09/26/14 10:31, Thomas Schwinge wrote: Hi! On Fri, 26 Sep 2014 10:18:32 -0600, Jeff Law wrote: On 09/26/14 07:23, Thomas Schwinge wrote: On Fri, 26 Sep 2014 12:04:45 +0400, Maxim Ostapenko wrote: Done in r215633. This is causing compiler warnings, respectively bootstrap errors: [...]

Re: [PATCH] Put all MAINTAINERS email addresses into <...>

2014-09-26 Thread Jeff Law
On 09/25/14 15:10, Jan-Benedict Glaw wrote: Hi! Resending this email. Seems some spam filter ate it due to the many email addresses... Following up on my suggestion to put all email addresses into <...> (cf. https://gcc.gnu.org/ml/gcc/2014-09/msg00298.html) here's an actual patch. Quite a mech

Re: [PATCH C++] - SD-6 Implementation Part 2 - __has_include macro and C++ language feature macros.

2014-09-26 Thread Jason Merrill
On 09/26/2014 10:20 AM, Ed Smith-Rowland wrote: On 09/25/2014 01:40 PM, Jason Merrill wrote: Don't we support attribute deprecated in C++11? We support [[gnu::deprecated]] in C++11 bit not [[deprecated]] until C++14. Hmm, that seems unnecessary. I'd allow it in C++11 as well, and *maybe* c

Re: [wwwdocs] Update C++1y status page now that C++14 is finished.

2014-09-26 Thread Gerald Pfeifer
On Wednesday 2014-09-24 08:24, Mike Stump wrote: >> C++14 is no longer the next standard, it's here, so update the project >> page. > Can we have a web doc person update the name of the page > (projects/cxx1y.html -> projects/cxx14.html) and add a redirect > as necessary? I'll do that over the w

Re: [debug-early] fix fortran regressions

2014-09-26 Thread Aldy Hernandez
On 09/26/14 01:29, Richard Biener wrote: So - please try dropping push_cfun as you set current_function_decl anyway. Excellent. Thanks for the clean-up suggestion. I am also including a small fix to squelch a use-before-def problem. Committing to branch. Whine if in violent opposition. Ald

[debug-early] reuse old DIE if it was dumped early

2014-09-26 Thread Aldy Hernandez
I'm not sure, but somewhere along the last few commits I caused a regression that inhibits libgfortran from building. Serves me right for forgetting to rebuild all the target libraries after each patch. For some instances of subprograms, we do not reuse the prexisting die, and we trigger the

Re: [jit] Add a test of using very long names

2014-09-26 Thread Mike Stump
On Sep 26, 2014, at 8:14 AM, David Malcolm wrote: > * jit.dg/test-long-names.c: New test case. > +/* 65KB */ > +#define NAME_LENGTH (65 * 1024) 65K was a tiny name back in 1999, 16M was a large name then. Today, 16M is tiny enough. And yeah, this was a customer bug report, just normal C

Re: FW: [PATCH] Fix PR preprocessor/58893 access to uninitialized memory

2014-09-26 Thread Jeff Law
On 09/26/14 06:21, Bernd Edlinger wrote: > >Hi, > >this patch fixes PR58893, which is an access to uninitialized memory, which may or may not crash in >linemap_resolve_location, or just print error messages with bogus location. > >When the first -include file is processed we have the case, where

Enable TBAA on anonymous types with LTO

2014-09-26 Thread Jan Hubicka
Hello, this is patch to preserve TBAA for anonymous types to LTO. The difference can be seen on the testcase: namespace { struct A {int a;}; struct B {int b;}; } struct A aa,*a=&aa; struct B bb,*b=&bb; void setA() { a->a=1; } void setB() { b->b=2; } int main() { asm("":"=r"(a),"=r"(b)

Re: Avoid privatization of TLS variables

2014-09-26 Thread Jan Hubicka
> On Fri, Sep 26, 2014 at 04:17:14AM +0200, Jan Hubicka wrote: > > I was building libreoffice with profile feedback and I run into a message > > > > cannot load any more object with static TLS > > > > that took me a while to track as I did not see where static TLS is comming > > out. > > Ian poi

[PATCH, rs6000, committed] Fix effective target in gcc.target/powerpc/pr63335.c

2014-09-26 Thread Bill Schmidt
Hi, I goofed on the effective target in the subject test, checking only if it was ok to produce VSX instructions, not whether we were running on hardware with VSX support. The latter is needed. This triggered a failure on the VSX-less regression tester. Fix committed as obvious. Thanks, Bill

Re: [jit] Add a test of using very long names

2014-09-26 Thread David Malcolm
On Fri, 2014-09-26 at 11:45 -0700, Mike Stump wrote: > On Sep 26, 2014, at 8:14 AM, David Malcolm > wrote: > > * jit.dg/test-long-names.c: New test case. > > > +/* 65KB */ > > +#define NAME_LENGTH (65 * 1024) > > 65K was a tiny name back in 1999, 16M was a large name then. Today, > 16M is t

C++ PATCH for abi_tag on builtin mangling abbreviations

2014-09-26 Thread Jason Merrill
We were ignoring abi_tags on the standard mangling abbreviations, such as Ss = std::string. This patch fixes that so that we append tags to the abbreviation, and add the result as a substitution. Tested x86_64-pc-linux-gnu, applying to trunk. commit 9e8c617e9135989151ba0a8d6f0d38c7913b2f23 Aut

Re: [Patch, MIPS] Add .note.GNU-stack section

2014-09-26 Thread Steve Ellcey
On Wed, 2014-09-10 at 10:15 -0700, Eric Christopher wrote: > > > On Wed, Sep 10, 2014 at 9:27 AM, wrote: > This works except you did not update the assembly files in > libgcc or glibc. We (Cavium) have the same patch in our tree > for a few released versions. > Mind jus

Re: [Patch] Switch elimination pass for PR 54742

2014-09-26 Thread Sebastian Pop
Jeff Law wrote: > On 08/21/14 04:30, Richard Biener wrote: > >>It turns Jeff's jump-threading code in to a strange franken-pass of bits and > >>pieces of detection and optimisation, and would need some substantial > >>reworking to fit in with Jeff's changes last Autumn, but if it is more > >>likely

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-26 Thread Andreas Schwab
t; } > + call co_reduce(vec([1,3,2]), red_f) ! { dg-error "Argument 'A' with > INTENT\\(INOUT\\) at .1. of the intrinsic subroutine co_min shall not have a > vector subscript" } > +end program test FAIL: gfortran.dg/coarray_collectives_9.f90 -O (test for errors,

[jit] Experimental in-process embedding of the gcc driver into the jit

2014-09-26 Thread David Malcolm
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: [...] > The code for compiling a .s file should: [...] > * use libiberty's pexecute to run subprocesses, not "system" (building up > a string to pass to the shell always looks like a security hole, though in > this case it may in fact be s

RFA: one more version of the patch for PR61360

2014-09-26 Thread Vladimir Makarov
I guess we achieved the consensus about the following patch to fix PR61360 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 The patch was successfully bootstrapped and tested (w/wo -march=amdfam10) on x86/x86-64. Is it ok to commit to trunk? 2014-09-26 Vladimir Makarov PR targe

Re: [PATCH 3/5] IPA ICF pass

2014-09-26 Thread Jan Hubicka
Hi, this is on ipa-icf-gimple.c @@ -2827,11 +2829,19 @@ cgraph_node::verify_node (void) { if (verify_edge_corresponds_to_fndecl (e, decl)) { - error ("edge points to wrong decl

Re: [PATCH IRA] update_equiv_regs fails to set EQUIV reg-note for pseudo with more than one definition

2014-09-26 Thread Jeff Law
On 09/26/14 07:57, Felix Yang wrote: Hi Jeff, Thanks for the suggestions. I updated the patch accordingly. 1. Both my employer(Huawei) and I have signed the copyright assignments with FSF. These assignments are already sent via post two days ago and hopefully should reach FSF

Re: [Patch, AArch64] Enable Address sanitizer and UB sanitizer

2014-09-26 Thread Andreas Schwab
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=34c65c4 * sanitizer_common/sanitizer_platform_limits_posix.h (__sanitizer___kernel_old_uid_t, __sanitizer___kernel_old_gid_t) [__aarch64__]: Define to unsigned short. --- libsanitizer/sanitizer_com

[PATCH] Fix finding default baseline symbols directory

2014-09-26 Thread Andreas Schwab
Tested on aarch64-suse-linux, where try_cpu=generic. Andreas. * configure.host: Use host_cpu, not try_cpu, to define default abi_baseline_pair. --- libstdc++-v3/configure.host | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libstdc++-v3/configure.host b/li

Re: [jit] Avoiding hardcoding "gcc"; supporting accelerators?

2014-09-26 Thread Joseph S. Myers
On Thu, 25 Sep 2014, David Malcolm wrote: > Should this have the $(exeext) suffix seen in Makefile.in? > $(target_noncanonical)-gcc-$(version)$(exeext) Depends on whether that's needed for the pex code to find it. > As for (B), would it make sense to "bake in" the path to the binary into > the

Re: [PATCH 1/n] OpenMP 4.0 offloading infrastructure

2014-09-26 Thread Joseph S. Myers
On Fri, 26 Sep 2014, Ilya Verbin wrote: > 2014-09-26 Bernd Schmidt > Thomas Schwinge > Ilya Verbin > Andrey Turetskiy > > * configure: Regenerate. > * configure.ac (--enable-as-accelerator-for) > (--enable-offload-targets): New configure op

Re: [PATCH 3/5] IPA ICF pass

2014-09-26 Thread Jan Hubicka
> diff --git a/gcc/ipa-icf.c b/gcc/ipa-icf.c > new file mode 100644 > index 000..f3472fe > --- /dev/null > +++ b/gcc/ipa-icf.c > @@ -0,0 +1,2841 @@ > +/* Interprocedural Identical Code Folding pass > + Copyright (C) 2014 Free Software Foundation, Inc. > + > + Contributed by Jan Hubicka and

[Patch, MIPS] Cleanup mips header files.

2014-09-26 Thread Steve Ellcey
I would like to do some cleanup on the mips configuration code, both to reduce the amount of duplicated code and to add better support for --with-arch, --with-endian, and --with-abi. As the first step in this work I would like to check in this patch that removes the linux64.h and gnu-user64.h hea

[fortran,patch] Emit code for some IEEE functions in the front-end

2014-09-26 Thread FX
Hi all, The attached patch improves our code generation for some of the IEEE_ARITHMETIC functions: some testing functions (is*) and some arithmetic (logb, rint, scalb, …). The interfaces are still present in the module file generated as part of the library (which allows, in particular, for accu

Re: [PATCH 3/5] IPA ICF pass

2014-09-26 Thread Jan Hubicka
> While a plain Firefox -flto build works fine. LTO/PGO build fails with: > > lto1: internal compiler error: in ipa_merge_profiles, at ipa-utils.c:540 > 0x7d6165 ipa_merge_profiles(cgraph_node*, cgraph_node*) > ../../gcc/gcc/ipa-utils.c:540 > 0xf10c41 ipa_icf::sem_function::merge(ipa_icf::

Move tail merging pass forward

2014-09-26 Thread Jan Hubicka
Hi, testcase in PR35545 shows case where profile feedback infrastructure does everything to make the testcase optimized (fully devirutalized) but it does not happen because tracer is run too late in the queue. Tail duplication in general is a pass enabling more optimizations to be done by forward

Fix pasto in ipa_polymorphic_call_context::restrict_to_inner_class

2014-09-26 Thread Jan Hubicka
Hi, this patch fixes ICE seen with testcase for PR62121 that is pasto in size checking. The GCC 4.9 issue is different and fixed in meantime. I will backport that change. Bootstrapped/regtested x86_64-linux, comitted. PR ipa/62121 * ipa-polymorphic-call.c (ipa_polymorphic_call

Re: [PATCH 3/5] IPA ICF pass

2014-09-26 Thread Markus Trippelsdorf
On 2014.09.27 at 01:27 +0200, Jan Hubicka wrote: > > While a plain Firefox -flto build works fine. LTO/PGO build fails with: > > > > lto1: internal compiler error: in ipa_merge_profiles, at ipa-utils.c:540 > > 0x7d6165 ipa_merge_profiles(cgraph_node*, cgraph_node*) > > ../../gcc/gcc/ipa-ut