Re: RFA: another patch to fix PR61360

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov vmaka...@redhat.com wrote: The previous patch to solve PR61360 fixed the problem in IRA (it was easier for me to do as I know the code well) https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 Although imo it was an ok fix, Richard

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-09-23 Thread Caroline Tice
Ok, your patch looks OK to me, but I can only approve the libvtv file changes. The changes in the other files also seem ok to me, but someone else will have to approve the modifications in them: gcc/config/i386/cygwin.h gcc/config/i386/mingw-w64.h gcc/config/i386/mingw32.h

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

2014-09-23 Thread Ilya Enkovich
2014-09-22 22:22 GMT+04:00 Jeff Law l...@redhat.com: On 09/21/14 06:34, Uros Bizjak wrote: On Fri, Sep 19, 2014 at 9:55 AM, Ilya Enkovich enkovich@gmail.com wrote: Here is an updated version of this patch. @@ -25000,10 +25082,32 @@ ix86_expand_call (rtx retval, rtx fnaddr, rtx

[Ping] [PATCH, 1/10] two hooks for conditional compare (ccmp)

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is attached for easy to apply. Thanks! -Zhenqiang -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen Sent: Monday, June 23, 2014 2:57 PM To: gcc-patches@gcc.gnu.org Subject: [PATCH, 1/10] two hooks

[Discard] [PATCH, 3/10] skip swapping operands used in ccmp

2014-09-23 Thread Zhenqiang Chen
The patch is discarded since I can not reproduce the issue with the latest trunk. Thanks! -Zhenqiang -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen Sent: Thursday, June 26, 2014 3:21 PM To: Richard

[Ping] [PATCH, 2/10] prepare ccmp

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is attached for easy to apply. Thanks! -Zhenqiang -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen Sent: Tuesday, July 01, 2014 1:08 PM To: Richard Earnshaw Cc: gcc-patches@gcc.gnu.org Subject:

[Ping] [PATCH, 5/10] aarch64: add ccmp operand predicate

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is attached for easy to apply. Thanks! -Zhenqiang -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen Sent: Monday, June 23, 2014 3:00 PM To: gcc-patches@gcc.gnu.org Subject: [PATCH, 5/10] aarch64:

[Ping] [PATCH, 4/10] expand ccmp

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is rebased and regenerated since [PATCH, 3/10] skip swapping operands used in ccmp is discarded. Please find the updated patch in attachment. Bootstrap and no make check regression on X86-64. Thanks! -Zhenqiang ChangeLog: 2014-09-23 Zhenqiang Chen zhenqiang.c...@linaro.org

[Ping] [PATCH, 6/10] aarch64: add ccmp CC mode

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is attached for easy to apply. Thanks! -Zhenqiang -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen Sent: Monday, June 23, 2014 3:00 PM To: gcc-patches@gcc.gnu.org Subject: [PATCH, 6/10] aarch64:

[Ping] [PATCH, 7/10] aarch64: add function to output ccmp insn

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is attached for easy to apply. Thanks! -Zhenqiang -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen Sent: Monday, June 23, 2014 3:00 PM To: gcc-patches@gcc.gnu.org Subject: [PATCH, 7/10] aarch64:

[Ping] [PATCH, 8/10] aarch64: ccmp insn patterns

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is attached for easy to apply. Move the cstorecc4 pattern from [PATCH, 10/10] aarch64: Handle ccmp in ifcvt to make it work with cmov to this patch, since cmov optimization does not depend on that patch. Thanks! -Zhenqiang ChangeLog: 2014-09-23 Zhenqiang Chen

[Ping] [PATCH, 9/10] aarch64: generate conditional compare instructions

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is attached for easy to apply. Bootstrap on AARCH64 qemu. Test cases: test_frame_*.c need update after the patch. Thanks! -Zhenqiang -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen Sent: Monday,

[Ping] [PATCH, 10/10] aarch64: Handle ccmp in ifcvt to make it work with cmov

2014-09-23 Thread Zhenqiang Chen
Ping? Patch is rebased and attached. Thanks! -Zhenqiang ChangeLog 2014-09-23 Zhenqiang Chen zhenqiang.c...@linaro.org Andrew Pinski apin...@cavium.com * ccmp.c (used_in_cond_stmt_p): Hande ? expr. * expr.c (expand_cond_expr_using_cmove): Handle CCmode. *

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-23 Thread Ilya Enkovich
2014-09-22 22:51 GMT+04:00 Uros Bizjak ubiz...@gmail.com: On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich enkovich@gmail.com wrote: On 19 Sep 18:21, Uros Bizjak wrote: On Fri, Sep 19, 2014 at 2:53 PM, Ilya Enkovich enkovich@gmail.com wrote: This patch adds i386 target hooks for

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

2014-09-23 Thread Maxim Ostapenko
On 09/19/2014 02:16 AM, Joseph S. Myers wrote: On Thu, 11 Sep 2014, Maxim Ostapenko wrote: In general, when cc1 or cc1plus ICE-es, we try to reproduce the bug by running compiler 3 times and comparing stderr and stdout on each attempt with respective ones that were gotten as the result of

Re: [PATCH, PR63229] fix assert in hash_table pch routines

2014-09-23 Thread Trevor Saunders
On Fri, Sep 19, 2014 at 11:23:11AM +0100, Richard Sandiford wrote: tsaund...@mozilla.com writes: diff --git a/gcc/ChangeLog b/gcc/ChangeLog index c048672..5b27aa8 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2014-09-12 Trevor Saunders tsaund...@mozilla.com + +

Re: [Ping] [PATCH, 1/10] two hooks for conditional compare (ccmp)

2014-09-23 Thread pinskia
On Sep 22, 2014, at 11:43 PM, Zhenqiang Chen zhenqiang.c...@arm.com wrote: Ping? Patch is attached for easy to apply. Note I have been using an earlier version of this patch set in house and not have found any issues with it. Thanks, Andrew Thanks! -Zhenqiang -Original

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 8:48 AM, Ilya Enkovich enkovich@gmail.com wrote: 2014-09-22 22:51 GMT+04:00 Uros Bizjak ubiz...@gmail.com: On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich enkovich@gmail.com wrote: On 19 Sep 18:21, Uros Bizjak wrote: On Fri, Sep 19, 2014 at 2:53 PM, Ilya

Re: Speedup int_bit_from_pos

2014-09-23 Thread Richard Biener
On Mon, 22 Sep 2014, Jan Hubicka wrote: Hi, it now seems to work, finaly :) Bootstrapped/regtested x86_64-linux, OK? * tree.c (int_bit_position): Move to... * tree.h (int_bit_position): ... here; implement using direct wide_int calculation instead of folding. Index:

Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

2014-09-23 Thread James Greenhalgh
On Mon, Sep 15, 2014 at 11:56:03AM +0100, Andrew Stubbs wrote: On 15/09/14 10:46, Richard Earnshaw wrote: Hmm, I wonder if arm_override_options should reject neon + (arch 7). Is this more to your taste? Is this really such a good idea? It causes carnage throughout the testsuite if you have

[PATCH][match-and-simplify] Also canonicalize comparison ops

2014-09-23 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2014-09-22 Richard Biener rguent...@suse.de * gimple-match-head.c (gimple_resimplify2): Also swap comparison operands. (gimple_simplify): Likewise. Index: gcc/gimple-match-head.c

[Patch] Cleanup widest_int_mode_for_size

2014-09-23 Thread James Greenhalgh
Hi, The comment on widest_int_mode_for_size claims that it returns the widest integer mode no wider than size. The implementation looks more like it finds the widest integer mode smaller than size. Everywhere it is used, the mode it is looking for is ultimately checked against an expected

[PATCH] Fix overactive reference removal with speculative devirtualization

2014-09-23 Thread Martin Jambor
Hi, code removing references to indirectly-inlined functions can remove a few too many when speculative devirtualization gets into the picture. Although we already handle the case when speculative devirtualization clones an edge with a CONST jump function by also cloning the appropriate

Re: [patch] Implement move semantics for iostreams

2014-09-23 Thread Andreas Schwab
errors) Excess errors: /daten/aranym/gcc/gcc-20140923/gcc/testsuite/g++.old-deja/g++.law/ctors10.C:16:46: error: call of overloaded 'basic_ostream(NULL)' is ambiguous Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7

Re: [patch] Implement move semantics for iostreams

2014-09-23 Thread Jonathan Wakely
++.law/ctors10.C -std=c++11 (test for excess errors) Excess errors: /daten/aranym/gcc/gcc-20140923/gcc/testsuite/g++.old-deja/g++.law/ctors10.C:16:46: error: call of overloaded 'basic_ostream(NULL)' is ambiguous Thanks, I'll adjust the new constructor.

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-09-23 Thread Jonathan Wakely
On 19/09/14 00:23 +0200, Patrick Wollgast wrote: Index: libstdc++-v3/acinclude.m4 === --- libstdc++-v3/acinclude.m4 (Revision 214408) +++ libstdc++-v3/acinclude.m4 (Arbeitskopie) @@ -2321,7 +2321,17 @@

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

2014-09-23 Thread Felix Yang
Hi, Attached please fined the updated patch. Cheers, Felix On Tue, Sep 23, 2014 at 10:48 AM, Yangfei (Felix) felix.y...@huawei.com wrote: On 09/22/14 08:40, Felix Yang wrote: Hi, I find that update_equiv_regs in ira.c sets the wrong EQUIV reg-note for pseudo with more than one

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

2014-09-23 Thread Felix Yang
Hi, Ignore the previous message. Attached please find the updated patch. Bootstrapped on x86_64-suse-linux. Please apply this patch if OK for trunk. Index: gcc/ChangeLog === --- gcc/ChangeLog(revision 215500) +++

Re: [PATCH] AArch64: Improve regmove_costs for 128-bit types

2014-09-23 Thread Marcus Shawcroft
On 15 September 2014 15:48, Wilco Dijkstra wdijk...@arm.com wrote: Hi, This patch improves the register move costs for 128-bit types. OK for commit? ChangeLog: 2014-09-15 Wilco Dijkstra wdijk...@arm.com * gcc/config/aarch64/aarch64.c (aarch64_register_move_cost): Add

[PATCH][match-and-simplify] Add predicate expressions

2014-09-23 Thread Richard Biener
As promised this allows you to define custom predicate expressions (or operators), like one modeled after tree-ssa-forwprop.c:lookup_logical_inverted_value: (match logical_inverted_value (bit_not truth_valued_p@0) (logical_inverted_value @0)) (match logical_inverted_value (eq @0

[PATCH i386 AVX512] [46/n] Add widening pmov.

2014-09-23 Thread Kirill Yukhin
Hello, Patch in the bottom extends pmov insns. 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 avx2_codev16qiv16hi2mask_name): Add masking. (define_insn

[PATCH i386 AVX512] [47/n] Add insert insn patterns.

2014-09-23 Thread Kirill Yukhin
Hello, Patch in the bottom extends insert insn patterns. It also removes some exapnds. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk?

Re: [PATCH] AArch64: Default to -fsched-pressure

2014-09-23 Thread Marcus Shawcroft
On 19 September 2014 18:28, Wilco Dijkstra wdijk...@arm.com wrote: This patch makes -fsched-pressure the default on AArch64, like on ARM. This improves performance and reduces codesize due to fewer unnecessary spills. OK for commit? ChangeLog: 2014-09-19 Wilco Dijkstra wdijk...@arm.com

[PATCH i386 AVX512] [48/n] Add shuffles (pd, 32x4, etc.).

2014-09-23 Thread Kirill Yukhin
Hello, Patch in the bottom extends more shuffle patterns. 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_sse2_shufpd, CODE_FOR_sse2_sse2_shufpd_mask,

Re: [patch] libstdc++/29988 Rb_Tree reuse allocated nodes

2014-09-23 Thread Jonathan Wakely
On 22/09/14 23:51 +0200, François Dumont wrote: New patch in a couple of day then. OK, thanks.

Re: Profile mode maintenance patch

2014-09-23 Thread Jonathan Wakely
On 21/09/14 23:29 +0200, François Dumont wrote: With all those modifications I have been able to run all testsuite in profile mode with success. I've looked over the patch and it looks fine. I don't know the details of the Profile Mode, so if you're happy that these changes are an

[PATCH i386 AVX512] [49/n] Add vpshuf[lh]w insn patterns.

2014-09-23 Thread Kirill Yukhin
Hello, 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

[PATCH i386 AVX512] [50/n] Add cvtps2 insn patterns.

2014-09-23 Thread Kirill Yukhin
Hello, Patch in the bottom extends truncation insn patterns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ *

Re: [AArch64] Re: [PATCH] Relax check against commuting XOR and ASHIFTRT in combine.c

2014-09-23 Thread Marcus Shawcroft
On 22 September 2014 12:16, Alan Lawrence alan.lawre...@arm.com wrote: Ok thanks Jeff. In that case I think I should draw this to the attention of the AArch64 maintainers to check the testsuite updates are OK before I commit...? OK with me. Cheers /Marcus

Re: [PATCH i386 AVX512] [49/n] Add vpshuf[lh]w insn patterns.

2014-09-23 Thread Kirill Yukhin
Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? Forgot to mention, that patch adds support for PSHUF[HL]W insns. -- Thanks, K

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-23 Thread Mark Wielaard
On Mon, Sep 22, 2014 at 10:59:38AM +0200, Andreas Arnez wrote: + sub_qual = TYPE_QUAL_RESTRICT; + if ((sub_quals ~sub_qual) != TYPE_UNQUALIFIED + get_qualified_type (type, sub_quals ~sub_qual) != NULL_TREE) +return true; + + sub_qual = TYPE_QUAL_CONST

[PATCH][match-and-simplify] Fix outer if parsing

2014-09-23 Thread Richard Biener
This restructures outer if parsing to maintain a stack of active ifs. This gets rid of all the reverse if stuff and fixes outer ifs on (match ...)es. What is left to be done is delaying 'for' lowering. Richard. 2014-09-23 Richard Biener rguent...@suse.de * genmatch.c

Re: [PATCH] Add D demangling support to libiberty

2014-09-23 Thread Iain Buclaw
On 19 September 2014 08:51, Iain Buclaw ibuc...@gdcproject.org wrote: On 4 August 2014 16:52, Ian Lance Taylor i...@google.com wrote: On Sun, Aug 3, 2014 at 11:12 AM, Iain Buclaw ibuc...@gdcproject.org wrote: - I haven't signed any copyright assignments to GCC. But I have papers from Donald

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-23 Thread Andreas Arnez
On Mon, Sep 22 2014, Andreas Arnez wrote: On Sat, Sep 20 2014, Mark Wielaard wrote: When adding DW_TAG_restrict_type I made a mistake when updating the code that handled types with multiple modifiers. This patch fixes it by putting the logic for finding the sub-qualified type in a separate

Re: [PATCH i386 AVX512] [46/n] Add widening pmov.

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 1:12 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Patch in the bottom extends pmov insns. 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

Re: [PATCH i386 AVX512] [47/n] Add insert insn patterns.

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 1:17 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, Patch in the bottom extends insert insn patterns. It also removes some exapnds. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ *

Re: [PATCH i386 AVX512] [48/n] Add shuffles (pd, 32x4, etc.).

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 1:22 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Patch in the bottom extends more shuffle patterns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c

Re: [PATCH i386 AVX512] [50/n] Add cvtps2 insn patterns.

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 1:32 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Patch in the bottom extends truncation 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

Re: [PATCH i386 AVX512] [49/n] Add vpshuf[lh]w insn patterns.

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 1:30 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: 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_PSHUFHW, UNSPEC_PSHUFLW.

Re: [PATCH, AArch64] Enable shrink wrap

2014-09-23 Thread Marcus Shawcroft
On 4 September 2014 15:14, Jiong Wang jiong.w...@arm.com wrote: this patch enabled stack shrink-wrap support on AArch64. no regression on aarch64-none-elf bare-metal. aarch64 bootstrap OK. ok to install? 2014-09-04 Renlin Lirenlin...@arm.com gcc/ * config/aarch64/aarch64.md (return):

[C++ Patch] PR 61857

2014-09-23 Thread Paolo Carlini
Hi, in this C++14 rejects valid issue, we reject the testcase with ‘x’ was not declared in this scope, because in cp_parser_initializer_list we tentatively try to parse what follows the '[' as a cp_parser_constant_expression. As far as I can see, we can easily solve the issue with the

Re: [PATCH, rs6000] Cleanup movsf/movsd/movdf/movdd constraints; Cleanup TImode boolean

2014-09-23 Thread David Edelsohn
On Mon, Sep 22, 2014 at 5:35 PM, Michael Meissner meiss...@linux.vnet.ibm.com wrote: I was working to support enhancing addressing modes in the the compiler that is indended to better support load fusion in power8, and to finally allow DFmode/SFmode scalar values in the traditional altivec

Re: [PATCH/RFC 15 / 14+2][RS6000] Remove vec_shl and (hopefully) fix vec_shr

2014-09-23 Thread David Edelsohn
On Thu, Sep 18, 2014 at 8:57 AM, Alan Lawrence alan.lawre...@arm.com wrote: Patch 12 of 14 (https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01475.html) will break bigendian targets implementing vec_shr. This is a PowerPC parallel of patch 13 of 14

Re: [gomp4] OpenACC routine directive

2014-09-23 Thread Thomas Schwinge
Hi Cesar! On Mon, 15 Sep 2014 16:35:02 -0700, Cesar Philippidis ce...@codesourcery.com wrote: This patch adds initial support for the OpenACC routine directive. Thanks! It's not complete just yet because it doesn't implement any of the optional clauses, except for the optional

Re: [patch] Implement move semantics for iostreams

2014-09-23 Thread Jonathan Wakely
On 23/09/14 11:48 +0200, Andreas Schwab wrote: FAIL: g++.old-deja/g++.law/ctors10.C -std=c++11 (test for excess errors) Excess errors: /daten/aranym/gcc/gcc-20140923/gcc/testsuite/g++.old-deja/g++.law/ctors10.C:16:46: error: call of overloaded 'basic_ostream(NULL)' is ambiguous Fixed

[PATCH][match-and-simplify][1/2] Delay for lowering

2014-09-23 Thread Richard Biener
This is a first cleanup step towards lowering for late, not during parsing. This intermediate step removes the e_operation indirection and makes parse_for temporarily insert operators it defines so we can avoid lazily accepting anything as user-defined and thus report proper error locations.

[jit] Fix segfault in ipa-pure-const.c during LTO

2014-09-23 Thread David Malcolm
Committed to branch dmalcolm/jit: In the initial commit on the JIT branch I moved gcc/ipa-pure-const.c: register_hooks to be a method of class pass_ipa_pure_const and the static local init_p within it to be member data of the class, to ensure that the pass can be rerun by separate invocations

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Ilya Enkovich
On 03 Sep 16:19, Vladimir Makarov wrote: On 2014-08-29 2:47 AM, Ilya Enkovich wrote: Seems your patch doesn't cover all cases. Attached is a modified patch (with your changes included) and a test where double constant is wrongly rematerialized. I also see in ira dump that there is still a

Re: [patch] Implement move semantics for iostreams

2014-09-23 Thread Rainer Orth
Jonathan Wakely jwak...@redhat.com writes: This adds move and swap functions to the iostream classes. Although this is a pretty large patch, it's a pure addition that only affects C++11 mode, and should have no effect on existing code because it won't be moving or swapping streams. I

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-23 Thread Ilya Enkovich
On 23 Sep 09:51, Richard Biener wrote: On Mon, 22 Sep 2014, Uros Bizjak wrote: On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich enkovich@gmail.com wrote: On 19 Sep 18:21, Uros Bizjak wrote: On Fri, Sep 19, 2014 at 2:53 PM, Ilya Enkovich enkovich@gmail.com wrote: +static

[PATCH][match-and-simplify] Caret diagnostics

2014-09-23 Thread Richard Biener
The following implements very simple caret diagnostics. A slow implementation is possible as we're only emitting fatal errors (and thus at most one). Committed. Richard. 2014-09-23 Richard Biener rguent...@suse.de * genmatch.c (error_cb): Implement simple caret diagnostics. Index:

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-23 Thread Richard Biener
On Tue, 23 Sep 2014, Ilya Enkovich wrote: On 23 Sep 09:51, Richard Biener wrote: On Mon, 22 Sep 2014, Uros Bizjak wrote: On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich enkovich@gmail.com wrote: On 19 Sep 18:21, Uros Bizjak wrote: On Fri, Sep 19, 2014 at 2:53 PM, Ilya

[PATCH] Introduce warning -Womp-default-scope

2014-09-23 Thread VandeVondele Joost
Attached patch introduces an optional warning for an OMP parallel/task/teams construct without explicit default(none). This would effectively catch the number one reason of easy-to-avoid OMP bugs in our project. Tested with check-fortran, full bootstrap check in progress. OK for trunk if this

Re: libsanitizer merge from upstream r218156

2014-09-23 Thread Jakub Jelinek
On Mon, Sep 22, 2014 at 11:44:57AM -0700, Konstantin Serebryany wrote: re-sending with the patch compressed: === gcc/testsuite/ChangeLog 2014-09-22 Kostya Serebryany k...@google.com Update to match the changed asan API. * asan.c (asan_global_struct): Update

Re: [patch] Implement move semantics for iostreams

2014-09-23 Thread Jonathan Wakely
On 23/09/14 15:58 +0200, Rainer Orth wrote: This patch broke Solaris bootstrap with Sun ld: when linking libstdc++.so, ld complains ld: fatal: libstdc++-symbols.ver-sun: 4520: symbol 'std::basic_ioschar, std::char_traitschar ::move(std::basic_ioschar, std::char_traitschar )': symbol version

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 3:54 PM, Ilya Enkovich enkovich@gmail.com wrote: Here is a patch which combines results of my and Vladimir's work on EBX enabling. It works OK for SPEC2000 and SPEC2006 on -Ofast + LTO. It passes bootstrap but there are few new failures in make check.

[jit] Don't hardcode documentation version/release

2014-09-23 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ChangeLog.jit: * docs/conf.py (__read_file): New helper function, for extracting... (gcc_BASEVER): New variable, read from BASE-VER in gcc src dir. (gcc_DEVPHASE): Likewise, from file DEV-PHASE. (gcc_DATESTAMP):

[PATCH] Avoid unnecessary DECL_UID -g/-g0 differences in Fortran FE (PR fortran/63331)

2014-09-23 Thread Jakub Jelinek
Hi! This patch changes get_array_descr_info langhook in Fortran FE not to create VAR_DECLs, because the langhook is only called with -g and not with -g0 and thus DECL_UIDs go out of sync too early. For VTA, we allow DECL_UID changes, but lhd_set_assembler_name sometimes emits DECL_UID into symbol

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote: use fixed EBX at least until we make sure pseudo PIC doesn't harm debug info generation. If we have such option then gcc.target/i386/pic-1.c and For debug info, it seems you are already handling this in delegitimize_address target

Re: [PATCH] Introduce warning -Womp-default-scope

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 02:16:58PM +, VandeVondele Joost wrote: Attached patch introduces an optional warning for an OMP parallel/task/teams construct without explicit default(none). This would effectively catch the number one reason of easy-to-avoid OMP bugs in our project. a) I don't

[committed] PR 63280: Misplaced destructor call

2014-09-23 Thread Richard Sandiford
My patch to fix a memory leak with target_globals accidentally introduced a double destructor call for default_target_ira. Normally this was benign because free_register_move_costs clears the freed pointers afterwards. However, with LTO -O3 that clearing could get inlined into the destructor and

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Vladimir Makarov
On 09/23/2014 02:07 AM, Uros Bizjak wrote: On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov vmaka...@redhat.com wrote: The previous patch to solve PR61360 fixed the problem in IRA (it was easier for me to do as I know the code well) https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-23 Thread Mark Wielaard
On Tue, 2014-09-23 at 14:17 +0200, Andreas Arnez wrote: Also note that the logic wouldn't scale too well for yet more qualifiers... Considering this, I've tried a different approach below. What do you think? Having support in tree.c instead of doing it by hand in dwarf2out.c is certainly

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 4:52 PM, Vladimir Makarov vmaka...@redhat.com wrote: On 09/23/2014 02:07 AM, Uros Bizjak wrote: On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov vmaka...@redhat.com wrote: The previous patch to solve PR61360 fixed the problem in IRA (it was easier for me to do as I

Re: [PATCH] Fix overactive reference removal with speculative devirtualization

2014-09-23 Thread Jan Hubicka
Hi, code removing references to indirectly-inlined functions can remove a few too many when speculative devirtualization gets into the picture. Although we already handle the case when speculative devirtualization clones an edge with a CONST jump function by also cloning the appropriate

RE: [PATCH] Introduce warning -Womp-default-scope

2014-09-23 Thread VandeVondele Joost
a) I don't like the option name, -Womp-no-default-clause would be IMHO better a bit related to b), I first had -Womp-no-default-none-clause but it becomes long and contains twice 'no' (also consider the no-omp-no-default... form). If you like that more, I'll make that change (see below)

Re: [C++ Patch] PR 61857

2014-09-23 Thread Jason Merrill
OK. Jason

[PATCH][ARM] Fix names of some rounding intrinsics, impement vrndx_f32 and vrndxq_f32

2014-09-23 Thread Kyrill Tkachov
Hi all, Some intrinsics had the wrong name (inconsistent with the NEON intrinsics spec). This patch fixes that and adds the vrndx_f32 and vrndxq_f32 intrinsics that were missing. These map down to vrintx.f32 NEON instructions (d and q forms). We already had builtins defined for them, just the

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-23 Thread Andreas Arnez
On Tue, Sep 23 2014, Mark Wielaard wrote: On Mon, Sep 22, 2014 at 10:59:38AM +0200, Andreas Arnez wrote: + sub_qual = TYPE_QUAL_RESTRICT; + if ((sub_quals ~sub_qual) != TYPE_UNQUALIFIED + get_qualified_type (type, sub_quals ~sub_qual) != NULL_TREE) +return true; + +

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Vladimir Makarov
On 09/23/2014 11:02 AM, Uros Bizjak wrote: On Tue, Sep 23, 2014 at 4:52 PM, Vladimir Makarov vmaka...@redhat.com wrote: On 09/23/2014 02:07 AM, Uros Bizjak wrote: Uros, my patch does not result in PR60704 (I tested it before submitting the patch). No, we didn't understand each other. The fix

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 5:02 PM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov vmaka...@redhat.com wrote: The previous patch to solve PR61360 fixed the problem in IRA (it was easier for me to do as I know the code well)

Re: [PATCH] Avoid unnecessary DECL_UID -g/-g0 differences in Fortran FE (PR fortran/63331)

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 05:20:34PM +0200, Tobias Burnus wrote: On 23.09.2014 16:27, Jakub Jelinek wrote: This patch changes get_array_descr_info langhook in Fortran FE not to create VAR_DECLs, because the langhook is only called with -g and not with -g0 and thus DECL_UIDs go out of sync too

Re: [PATCH][ARM] Fix names of some rounding intrinsics, impement vrndx_f32 and vrndxq_f32

2014-09-23 Thread Kyrill Tkachov
On 23/09/14 16:07, Kyrill Tkachov wrote: Hi all, Some intrinsics had the wrong name (inconsistent with the NEON intrinsics spec). This patch fixes that and adds the vrndx_f32 and vrndxq_f32 intrinsics that were missing. For reference, the NEON intrinsics spec can be found at:

Re: parallel check output changes?

2014-09-23 Thread Richard Sandiford
Segher Boessenkool seg...@kernel.crashing.org writes: On Thu, Sep 18, 2014 at 01:44:55PM -0500, Segher Boessenkool wrote: I am testing a patch that is just diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py index cccbfd3..3781423 100644 ---

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 5:22 PM, Vladimir Makarov vmaka...@redhat.com wrote: You are right constrain_operands is not upto LRA possibilities and we should make the following change: Index: recog.c === --- recog.c (revision

Re: parallel check output changes?

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 04:33:19PM +0100, Richard Sandiford wrote: FWIW, the \S+ thing was deliberate. When one test is run multiple times with different options, those options aren't necessarily tried in alphabetical order. The old sh/awk script therefore used just the test name as the key

Re: Speedup int_bit_from_pos

2014-09-23 Thread Richard Sandiford
Jan Hubicka hubi...@ucw.cz writes: +/* Like bit_position, but return as an integer. It must be representable in + that way (since it could be a signed value, we don't have the + option of returning -1 like int_size_in_byte can. */ + +static inline HOST_WIDE_INT +int_bit_position

[jit] Fix toyvm.c to run within the testsuite

2014-09-23 Thread David Malcolm
Committed to branch dmalcolm/jit: I wrote toyvm to be run standalone, but with 63ff70786e01bd6265e9e7ac1052e4cb8fe5dd38 it's also being run by jit.exp in the testsuite. Fix it up so that it meaningfully exercises all this code when run in this way. This brings the number of expected passes in

Re: [PATCH, Pointer Bounds Checker 23/x] Function split

2014-09-23 Thread Jeff Law
On 09/22/14 00:40, Ilya Enkovich wrote: Bounds don't have to vary for different pointers. E.g. p and p + 1 always have equal bounds. In this particular case we have function pointers and all of them have default bounds. OK. It looked a bit odd and I wanted to make sure there wasn't

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jeff Law
On 09/23/14 08:23, Uros Bizjak wrote: On Tue, Sep 23, 2014 at 3:54 PM, Ilya Enkovich enkovich@gmail.com wrote: Here is a patch which combines results of my and Vladimir's work on EBX enabling. It works OK for SPEC2000 and SPEC2006 on -Ofast + LTO. It passes bootstrap but there are few

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Petr Machata
Jakub Jelinek ja...@redhat.com writes: look at the sizes of .debug_info/.debug_loc sections with/without the patch, or use the locstat utility from elfutils Not actually part of elfutils, but available either here: https://github.com/pmachata/dwlocstat ... or packaged in Fedora.

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jeff Law
On 09/23/14 08:34, Jakub Jelinek wrote: On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote: use fixed EBX at least until we make sure pseudo PIC doesn't harm debug info generation. If we have such option then gcc.target/i386/pic-1.c and For debug info, it seems you are already

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-23 Thread Steven Bosscher
On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law l...@redhat.com wrote: On 09/19/14 13:36, Ilya Enkovich wrote: Hi, During my work on enabling pseudo PIC register I've found that cfg cleaunp may remove lables with LABEL_PRESERVE_P set to 1. In my case I generated SET_RIP during expand pass and

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 10:00:00AM -0600, Jeff Law wrote: On 09/23/14 08:34, Jakub Jelinek wrote: On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote: use fixed EBX at least until we make sure pseudo PIC doesn't harm debug info generation. If we have such option then

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-23 Thread Jeff Law
On 09/23/14 10:01, Steven Bosscher wrote: On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law l...@redhat.com wrote: On 09/19/14 13:36, Ilya Enkovich wrote: Hi, During my work on enabling pseudo PIC register I've found that cfg cleaunp may remove lables with LABEL_PRESERVE_P set to 1. In my case I

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jeff Law
On 09/23/14 10:03, Jakub Jelinek wrote: On Tue, Sep 23, 2014 at 10:00:00AM -0600, Jeff Law wrote: On 09/23/14 08:34, Jakub Jelinek wrote: On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote: use fixed EBX at least until we make sure pseudo PIC doesn't harm debug info generation. If

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Richard Biener
On September 23, 2014 5:33:35 PM CEST, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Sep 23, 2014 at 5:22 PM, Vladimir Makarov vmaka...@redhat.com wrote: You are right constrain_operands is not upto LRA possibilities and we should make the following change: Index: recog.c

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-23 Thread Jeff Law
On 09/23/14 08:10, Ilya Enkovich wrote: Please use fold_convert (size_ptr, build_fold_addr_expr (var)). Is 'var' always accessed via a size_t effective type? Watch out for TBAA issues if not. (if it is, why is 'var' not of type size_t or size_t[]?) var has pointer bounds type. I have to

[GOOGLE] Fix dynamic instrumentation patching for cold sections

2014-09-23 Thread Teresa Johnson
This patch fixes the support for patching for instrumentation so that it works with function splitting, by using the correct helper method for locating the current section. Added a couple tests cloned from existing function splitting tests that I confirmed expose the problem. Passes regression

Re: [GOOGLE] Fix dynamic instrumentation patching for cold sections

2014-09-23 Thread Xinliang David Li
ok. David On Tue, Sep 23, 2014 at 10:27 AM, Teresa Johnson tejohn...@google.com wrote: This patch fixes the support for patching for instrumentation so that it works with function splitting, by using the correct helper method for locating the current section. Added a couple tests cloned

  1   2   >