Re: std::regex: inserting std::wregex to std::vector loses some std::wregex values

2014-09-24 Thread Tim Shen
On Tue, Sep 16, 2014 at 5:28 PM, Tim Shen tims...@google.com wrote: So I'll change the patch to move _M_traits to _NFA, and add a new basic_regex::_M_loc member. Here it is :). Bootstrapped and tested with debug flag. Should the abi compatible fix be another patch for branch 4.9? In which

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Ilya Enkovich
2014-09-23 20:06 GMT+04:00 Jeff Law l...@redhat.com: 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

Re: Enable EBX for x86 in 32bits PIC code

2014-09-24 Thread Ilya Enkovich
2014-09-23 20:10 GMT+04:00 Jeff Law l...@redhat.com: 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

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

2014-09-24 Thread Ilya Enkovich
2014-09-23 22:01 GMT+04:00 Jeff Law l...@redhat.com: On 09/23/14 00:31, Ilya Enkovich wrote: I did this change a couple of years ago and don't remember exactly what problem was caused by PARALLEL. But from my comment it seems parallel lead to values in BND0 and BND1 not to be actually

Re: [gomp4] OpenACC wait directive

2014-09-24 Thread Ilmir Usmanov
Hi Cesar! Thank you for the patch! On 24.09.2014 02:29, Cesar Philippidis wrote: This patch adds support for the async clause in the wait directive in fortran. It should be pretty straight forward. The fortran FE already supports the wait directive, but the async clause was introduced to the

Re: [PATCH, Pointer Bounds Checker 22/x] Inline

2014-09-24 Thread Ilya Enkovich
2014-09-23 23:55 GMT+04:00 Jeff Law l...@redhat.com: On 08/18/14 09:35, Ilya Enkovich wrote: Here is an updated version. Thanks, Ilya -- 2014-08-15 Ilya Enkovich ilya.enkov...@intel.com * ipa-inline.c (early_inliner): Check edge has summary allocated. * tree-inline.c:

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

2014-09-24 Thread Richard Biener
On Tue, Sep 16, 2014 at 12:24 PM, Thomas Preud'homme thomas.preudho...@arm.com wrote: Hi all, The fix for PR61306 disabled bswap when a sign extension is detected. However this led to a test case regression (and potential performance regression) in case where a sign extension happens but

Re: [PATCH, testsuite]: PR 58757: Check for FP denormal values without triggering denormal exceptions

2014-09-24 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 8:40 PM, Marc Glisse marc.gli...@inria.fr wrote: Attached patch avoids triggering denormal exceptions when FP insns are used to check for non-zero denormal values. But I thought the point of the test was to verify that the compiler's understanding of existence of

Re: [PATCH, Pointer Bounds Checker 19/x] Support bounds in expand

2014-09-24 Thread Ilya Enkovich
2014-09-24 0:58 GMT+04:00 Jeff Law l...@redhat.com: On 06/05/14 08:46, Ilya Enkovich wrote: 2014-06-05 Ilya Enkovich ilya.enkov...@intel.com * calls.c: Include tree-chkp.h, rtl-chkp.h, bitmap.h. (arg_data): Add fields special_slot, pointer_arg and pointer_offset.

[PATCH i386 AVX512] [51/n] Add pd2dq and dq2pd converts.

2014-09-24 Thread Kirill Yukhin
Hello, Patch in the bottom adds support for pd2dq and dq2pd conversions. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c (avx512f_ufix_notruncv8dfv8si_mask_round): Rename to ...

[PATCH i386 AVX512] [52/n] Add convert ps2pd and ps2dq.

2014-09-24 Thread Kirill Yukhin
Hello, Patch in the bottom adds support for ps2dq and ps2pd conversions. 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_CVTINT2MASK. (define_insn

[PATCH] Fix asan optimization for aligned accesses. (PR sanitizer/63316)

2014-09-24 Thread Jakub Jelinek
On Tue, Sep 02, 2014 at 07:09:50PM +0400, Marat Zakirov wrote: Here's a simple optimization patch for Asan. It stores alignment information into ASAN_CHECK which is then extracted by sanopt to reduce number of and 0x7 instructions for sufficiently aligned accesses. I checked it on linux kernel

Re: libsanitizer merge from upstream r218156

2014-09-24 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 11:03:55AM -0700, Konstantin Serebryany wrote: OT, will you please look at the underaligned asan malloc etc.? GCC assumes that even malloc (1) or malloc (7) is sizeof (void *) aligned on Linux (and can and will assume 2 * sizeof (void *) alignment hopefully soon).

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Steven Bosscher
On Wed, Sep 24, 2014 at 8:41 AM, Ilya Enkovich wrote: 2014-09-23 20:06 GMT+04:00 Jeff Law: On 09/23/14 10:01, Steven Bosscher wrote: Are you sure this patch is necessary, and is not just papering over another problem? In the past, all cases I've seen where labels were removed inadvertently

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Jonathan Wakely
On 22/09/14 14:35 +0100, Jonathan Wakely wrote: This adds move and swap functions to the iostream classes. This fixes a silly typo. Tested x86_64-linux, committed to trunk. commit acaef9854dff5f37d86b80fc8236df5fd90b0ca5 Author: Jonathan Wakely jwak...@redhat.com Date: Wed Sep 24 10:10:28

Re: [PATCH 1/14][AArch64] Temporarily remove aarch64_gimple_fold_builtin code for reduction operations

2014-09-24 Thread Marcus Shawcroft
On 18 September 2014 12:45, Alan Lawrence alan.lawre...@arm.com wrote: The gimple folding ties the AArch64 backend to the tree representation of the midend via the neon intrinsics. This code enables constant folding of Neon intrinsics reduction ops, so improves performance, but is not necessary

Re: [PATCH 4/14][AArch64] Use new reduc_plus_scal optabs, inc. for __builtins

2014-09-24 Thread Marcus Shawcroft
On 18 September 2014 12:59, Alan Lawrence alan.lawre...@arm.com wrote: This migrates AArch64 over to the new optab for 'plus' reductions, i.e. so the define_expands produce scalars by generating a MOV to a GPR. Effectively, this moves the vget_lane inside every arm_neon.h intrinsic, into the

Re: [PATCH 5/14][AArch64] Use new reduc_[us](min|max)_scal optabs, inc. for builtins

2014-09-24 Thread Marcus Shawcroft
On 18 September 2014 13:02, Alan Lawrence alan.lawre...@arm.com wrote: Similarly to the previous patch (r/2205), this migrates AArch64 to the new reduce-to-scalar optabs for min and max. For consistency we apply the same treatment to the smax_nan and smin_nan patterns (used for __builtins),

Re: [PATCH 6/14][AArch64] Restore gimple_folding of reduction intrinsics

2014-09-24 Thread Marcus Shawcroft
On 18 September 2014 13:05, Alan Lawrence alan.lawre...@arm.com wrote: This gives us back the constant-folding of the neon-intrinsics that was removed in the first patch, but is now OK for bigendian too. bootstrapped on aarch64-none-linux-gnu. check-gcc on aarch64-none-elf and

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Ilya Enkovich
2014-09-24 13:30 GMT+04:00 Steven Bosscher stevenb@gmail.com: On Wed, Sep 24, 2014 at 8:41 AM, Ilya Enkovich wrote: 2014-09-23 20:06 GMT+04:00 Jeff Law: On 09/23/14 10:01, Steven Bosscher wrote: Are you sure this patch is necessary, and is not just papering over another problem? In the

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Jakub Jelinek
On Wed, Sep 24, 2014 at 10:40:09AM +0100, Jonathan Wakely wrote: On 22/09/14 14:35 +0100, Jonathan Wakely wrote: This adds move and swap functions to the iostream classes. This fixes a silly typo. Tested x86_64-linux, committed to trunk. commit acaef9854dff5f37d86b80fc8236df5fd90b0ca5

Re: Fix libgomp crash without TLS (PR42616)

2014-09-24 Thread Varvara Rainchik
*Ping* 2014-09-19 15:41 GMT+04:00 Varvara Rainchik varvara.s.rainc...@gmail.com: I've corrected my patch accordingly to what you said. To diffirentiate second case in destructor I've added pthread_setspecific (gomp_tls_key, NULL) at the end of gomp_thread_start. So, destructor can simply skip

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

2014-09-24 Thread Richard Biener
This delays for lowering by recording fors to apply in simplify similar to how we record ifs. Bootstrapped on x86_64-unknown-linux-gnu. Richard. 2014-09-24 Richard Biener rguent...@suse.de * genmatch.c (id_base): Derive from typed_noop_remove. (struct user_id): New id_base

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Marek Polacek
On Wed, Sep 24, 2014 at 12:01:13PM +0200, Jakub Jelinek wrote: - if (!__lhs_local !__lhs_local) + if (!__lhs_local !__rhs_local) std::swap(_M_word, __rhs._M_word); else { Wouldn't this be something for a (non-Wall?) warning? I mean if or || contains

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Steven Bosscher
On Wed, Sep 24, 2014 at 11:57 AM, Ilya Enkovich wrote: 2014-09-24 13:30 GMT+04:00 Steven Bosscher : Description of LABEL_PRESERVE_P says label that should always be considered to be needed. It's more specific than that, really: @item LABEL_PRESERVE_P (@var{x}) In a @code{code_label} or

[PATCH] Provide global var location info for asan

2014-09-24 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 11:03:55AM -0700, Konstantin Serebryany wrote: (asan_add_global): Ditto. I'll handle creation of location aggregates as follow-up. Here it is, only lightly tested so far: int a = 1; int b = 2; int c = 3; int * foo (int x) { return x ? b : c; } int main

[PATCH][match-and-simplify] Cleanup operator arity related diagnostics

2014-09-24 Thread Richard Biener
The following removes a bunch of code dealing with late verifying of operator presence and matching arity. This can now all be verified at parsing, giving proper locations and operator names. Bootstrap pending. Richard. 2014-09-24 Richard Biener rguent...@suse.de * genmatch.c

[PATCH][match-and-simplify] Remove outlining of C exprs

2014-09-24 Thread Richard Biener
It no longer works in the face of (with { } which would need to pass down all named temporaries. Instead we can simply inline all C exprs now that we pass 'output' to all gen_transform calls. Boostrap pending on x86_64-unknown-linux-gnu. Richard. 2014-09-24 Richard Biener

RE: RFA: another patch to fix PR61360

2014-09-24 Thread Gopalasubramanian, Ganesh
The r-x alternative results in vector decoding on amdfam10. This is AMD-speak for microcoded instructions, and AMD optimization manual strongly recommends avoiding them. I have CC'd Ganesh, maybe he can provide more relevant data on the performance impact. Thanks Uros! Yes, the AMD SWOG

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

2014-09-24 Thread Felix Yang
Hi Jeff, Thanks for the comments. I updated the patch adding some enhancements. Bootstrapped on x86_64-suse-linux. Please apply this patch if OK for trunk. Three points: 1. For multiple-set register, it is not qualified to have a equiv note once it is marked by no_equiv. The

[PATCH, bootstrap PR63235] Fix bootstrap.

2014-09-24 Thread Kirill Yukhin
Hello, Patch in the bottom fixes bootstrap (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63235) gcc/ * varpool.c (varpool_node::add): Pass decl attributes to lookup_attribute. Is it ok for trunk? -- Thanks, K diff --git a/gcc/varpool.c b/gcc/varpool.c index 8001c93..3761f14

Re: [PATCH, bootstrap PR63235] Fix bootstrap.

2014-09-24 Thread Jakub Jelinek
On Wed, Sep 24, 2014 at 04:16:50PM +0400, Kirill Yukhin wrote: Hello, Patch in the bottom fixes bootstrap (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63235) gcc/ * varpool.c (varpool_node::add): Pass decl attributes to lookup_attribute. Is it ok for trunk? Ok, thanks.

Re: [PATCH, testsuite]: PR 58757: Check for FP denormal values without triggering denormal exceptions

2014-09-24 Thread Marc Glisse
On Wed, 24 Sep 2014, Uros Bizjak wrote: However, alpha *does* support all IEEE features, the only problem is in its default model, which is for some reason High-Performance IEEE-Format Arithmetic (please see alpha AHB [1], section 4.7.6.5). This model does not require the overhead of an

Re: [PATCH][AArch64] Use __aarch64_vget_lane* macros for getting the lane in some lane multiply intrinsics

2014-09-24 Thread Kyrill Tkachov
Must have slipped through the cracks. https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00586.html Ping? Thanks, Kyrill On 08/09/14 11:29, Kyrill Tkachov wrote: Hi all, The included testcase currently ICEs at -O0 because vget_lane_f64 is a function, so if it's properly called with a constant

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Ilya Enkovich
2014-09-24 14:35 GMT+04:00 Steven Bosscher stevenb@gmail.com: On Wed, Sep 24, 2014 at 11:57 AM, Ilya Enkovich wrote: 2014-09-24 13:30 GMT+04:00 Steven Bosscher : Description of LABEL_PRESERVE_P says label that should always be considered to be needed. It's more specific than that,

Re: [PATCH 0/10] OpenACC 2.0 support for libgomp

2014-09-24 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 07:17:25PM +0100, Julian Brown wrote: The upcoming patch series constitutes our current (still in-progress) implementation of run-time support for OpenACC 2.0 in libgomp. We've tried to build on top of the (also currently WIP) support for OpenMP 4.0's target construct,

Re: [PATCH][AArch64] Use __aarch64_vget_lane* macros for getting the lane in some lane multiply intrinsics

2014-09-24 Thread Marcus Shawcroft
On 8 September 2014 11:29, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, The included testcase currently ICEs at -O0 because vget_lane_f64 is a function, so if it's properly called with a constant argument but without constant propagation it will not be recognised as constant, causing

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

2014-09-24 Thread Andreas Arnez
On Tue, Sep 23 2014, Mark Wielaard wrote: This certainly looks nicer than how I wrote it. It took me a while (again) to realize why this works. We rely on the fact that earlier in the function a match would have been found if there was already a fully qualified type available. So here we know

[PATCH, rs6000] Teach analyze_swaps to avoid vec_ste

2014-09-24 Thread Bill Schmidt
Hi, The analyze_swaps pass performs special handling on certain non-swapping loads and stores so that computations involving them can still be optimized. However, the intent was to avoid this for lvx, stvx, lve*, and stve*. The existing logic excludes these by looking for a PARALLEL as the rtx

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Steven Bosscher
On Wed, Sep 24, 2014 at 2:30 PM, Ilya Enkovich wrote: I didn't generate references separately from label. Now I found an old patch and a test where this problem appeared. In this patch I moved set_rip generation currently performed in ix86_expand_prologue into expand pass. And I got

[PATCH i386 AVX512] [53/n] Update vec_setmode_0 pattern constraints.

2014-09-24 Thread Kirill Yukhin
Hello, Patch in the bottom extends to EVEX constraints of vec_setmode_0 insn pattern. 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 vec_setmode_0): Add EVEX version. -- Thanks, K diff --git

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Ilya Enkovich
2014-09-24 16:47 GMT+04:00 Steven Bosscher stevenb@gmail.com: On Wed, Sep 24, 2014 at 2:30 PM, Ilya Enkovich wrote: I didn't generate references separately from label. Now I found an old patch and a test where this problem appeared. In this patch I moved set_rip generation currently

[PATCH i386 AVX512] [54/n] Add mov[dlh]dup insns support.

2014-09-24 Thread Kirill Yukhin
Hello, patch in the bottom introduces support for vmov[dlh]dup 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 avx_movshdup256mask_name): Add masking. (define_insn

[PATCH i386 AVX512] [55/n] Extend `perm' insn patterns.

2014-09-24 Thread Kirill Yukhin
Hello, Patch in the bottom extends `perm' 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 avx2_avx512f_permmode): Rename to ... (define_expand avx2_avx512bw_permmode):

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

2014-09-24 Thread Jonathan Wakely
C++14 is no longer the next standard, it's here, so update the project page. Committed to cvs. Index: projects/cxx1y.html === RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cxx1y.html,v retrieving revision 1.15 retrieving revision 1.16

[wwwdocs] Update libstdc++ section of gcc-5/changes.html

2014-09-24 Thread Jonathan Wakely
I forgot to send this patch a couple of days ago. Document recent libstdc++ changes on trunk. Also tweak the 4.9 notes slightly for consistency. Committed to cvs. Index: gcc-5/changes.html === RCS file:

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

2014-09-24 Thread Jakub Jelinek
On Wed, Sep 24, 2014 at 02:40:14PM +0200, Andreas Arnez wrote: A few style aspects I'm not sure about: * Is it OK to use __builtin_popcount in tree.c? Definitely not, you can use popcount_hwi instead, which for GCC host compiler (= 3.4) will use __builtin_popcount*, otherwise fallback to a

Re: [PATCH, rs6000] Teach analyze_swaps to avoid vec_ste

2014-09-24 Thread David Edelsohn
On Wed, Sep 24, 2014 at 8:46 AM, Bill Schmidt wschm...@linux.vnet.ibm.com wrote: Hi, The analyze_swaps pass performs special handling on certain non-swapping loads and stores so that computations involving them can still be optimized. However, the intent was to avoid this for lvx, stvx,

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

2014-09-24 Thread Ilya Enkovich
2014-09-24 11:05 GMT+04:00 Ilya Enkovich enkovich@gmail.com: 2014-09-23 22:01 GMT+04:00 Jeff Law l...@redhat.com: On 09/23/14 00:31, Ilya Enkovich wrote: I did this change a couple of years ago and don't remember exactly what problem was caused by PARALLEL. But from my comment it seems

[GOOGLE] Fix new tests

2014-09-24 Thread Teresa Johnson
The new tests added for -mpatch-functions-for-instrumentation did not correctly restrict themselves to x86_64 since tree-prof.exp doesn't support dg-do. Work around this by using target selectors on the dg-options. I apply the -mpatch and related options only if it is x86_64, otherwise it simply

Re: [patch] Implement move semantics for iostreams

2014-09-24 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: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-24 Thread Andreas Arnez
On Wed, Sep 24 2014, Jakub Jelinek wrote: On Wed, Sep 24, 2014 at 02:40:14PM +0200, Andreas Arnez wrote: A few style aspects I'm not sure about: * Is it OK to use __builtin_popcount in tree.c? Definitely not, you can use popcount_hwi instead, which for GCC host compiler (= 3.4) will use

Re: std::regex: inserting std::wregex to std::vector loses some std::wregex values

2014-09-24 Thread Jonathan Wakely
On 23/09/14 23:11 -0700, Tim Shen wrote: So I'll change the patch to move _M_traits to _NFA, and add a new basic_regex::_M_loc member. Here it is :). Bootstrapped and tested with debug flag. OK for trunk - thanks.

Re: [PATCH] Fix asan optimization for aligned accesses. (PR sanitizer/63316)

2014-09-24 Thread ygribov
BTW, I've noticed that perhaps using BIT_AND_EXPR for the (shadow != 0) ((base_addr 7) + (real_size_in_bytes - 1) = shadow) tests isn't best, maybe we could get better code if we expanded it as (shadow != 0) ((base_addr 7) + (real_size_in_bytes - 1) = shadow) (i.e. an extra basic

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Steven Bosscher
On Wed, Sep 24, 2014 at 2:51 PM, Ilya Enkovich wrote: 2014-09-24 16:47 GMT+04:00 Steven Bosscher : It is not a control flow instruction. It copies value of instruction pointer into a general purpose register. Therefore REG_LABEL_OPERAND seems to be correct. OK - sorry for being a bit slow on

Re: [PATCH] Fix asan optimization for aligned accesses. (PR sanitizer/63316)

2014-09-24 Thread ygribov
AFAIK LLVM team recently got some 1% on SPEC from this. On x64 that is. -- View this message in context: http://gcc.1065356.n5.nabble.com/Re-please-verify-my-mail-to-community-tp1066917p1073371.html Sent from the gcc - patches mailing list archive at Nabble.com.

Re: Updated no_reorder patchkit

2014-09-24 Thread Martin Liška
On 09/16/2014 05:15 AM, Andi Kleen wrote: This version addresses earlier comments and has an updated testsuite (still no LTO tests however). The assembler statements also no stay in order with ordered statements. It doesn't disable sorting of paritions with ordered symbols. I think that's an

Re: Updated no_reorder patchkit

2014-09-24 Thread Jan Hubicka
There's missing DECL_ATTRIBUTES in varpool.c in lookup_attribute call. Ready for trunk? OK, thanks Honza Martin diff --git a/gcc/varpool.c b/gcc/varpool.c index 8001c93..3761f14 100644 --- a/gcc/varpool.c +++ b/gcc/varpool.c @@ -449,7 +449,7 @@ varpool_node::add (tree decl)

Re: Updated no_reorder patchkit

2014-09-24 Thread Jakub Jelinek
On Wed, Sep 24, 2014 at 04:16:44PM +0200, Martin Liška wrote: On 09/16/2014 05:15 AM, Andi Kleen wrote: This version addresses earlier comments and has an updated testsuite (still no LTO tests however). The assembler statements also no stay in order with ordered statements. It doesn't

Re: Updated no_reorder patchkit

2014-09-24 Thread Martin Liška
On 09/24/2014 04:17 PM, Jan Hubicka wrote: There's missing DECL_ATTRIBUTES in varpool.c in lookup_attribute call. Ready for trunk? OK, thanks Honza Ah, it has been fixed in r215552. Martin Martin diff --git a/gcc/varpool.c b/gcc/varpool.c index 8001c93..3761f14 100644 ---

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

2014-09-24 Thread Martin Liška
On 06/13/2014 12:26 PM, mliska wrote: Hi, this small patch prepares remaining needed infrastructure for the new pass. Changelog: 2014-06-13 Martin Liska mli...@suse.cz Honza Hubicka hubi...@ucw.cz * ipa-utils.h (polymorphic_type_binfo_p): Function marked external

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Chen Gang
Hello Michael: Firstly, thank you very much for always providing your aid to me for microblaze. At present, after try testsuite, the result is much better than my original trying, please help check the result: is it enough for our microblaze testsuite (can we say it pass checking)? Current

[jit] Update the various *_c_finalize functions

2014-09-24 Thread David Malcolm
Joseph - thanks for looking through the jit diff. I plan to fix the issues you raise as a series of separate patches. Here's the first: On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: Various *_finalize functions are missing comments explaining their semantics. Also the return

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Ilya Enkovich
2014-09-24 17:50 GMT+04:00 Steven Bosscher stevenb@gmail.com: On Wed, Sep 24, 2014 at 2:51 PM, Ilya Enkovich wrote: 2014-09-24 16:47 GMT+04:00 Steven Bosscher : It is not a control flow instruction. It copies value of instruction pointer into a general purpose register. Therefore

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Rainer Orth
Hi Jonathan, 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 )':

PING: Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-24 Thread Aldy Hernandez
On 09/18/14 12:39, Aldy Hernandez wrote: Yeah, sure, either way it's a good cleanup ;). No strong opinions. Though I think true/false are the way we want folks to write new code. Given that's the long term direction, might as well fix that nit for DECL_ABSTRACT_P. Alright... fixed. OK?

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

2014-09-24 Thread Ilya Enkovich
2014-09-23 21:17 GMT+04:00 Jeff Law l...@redhat.com: 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

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

2014-09-24 Thread Jason Merrill
On 09/23/2014 06:53 PM, Mark Wielaard wrote: And for the default case (gcc doesn't create type sections by default) the optimization is useful. I'm skeptical. These DIEs are very small, and I wouldn't expect a hole in the qualifier space like this to come up that often. Jason

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Jonathan Wakely
On 24/09/14 16:38 +0200, Rainer Orth wrote: Hi Jonathan, 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

Re: parallel check output changes?

2014-09-24 Thread Andrew MacLeod
On 09/23/2014 11:33 AM, Richard Sandiford wrote: Segher Boessenkoolseg...@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

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

2014-09-24 Thread Jan Hubicka
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 mli...@suse.cz * cgraph.c (cgraph_node::release_body): New argument keep_arguments introduced.

Re: [PATCH 2/14][Vectorizer] Make REDUC_xxx_EXPR tree codes produce a scalar result

2014-09-24 Thread Alan Lawrence
So it looks like patches 1-6 (reduc_foo) are relatively close to final, and given these fix PR/61114, I'm gonna try to land these while working on a respin of the second half (vec_shr)...(summary: yes I like the vec_perm idea too, but the devil is in the detail!) However my CompileFarm

[AArch64] Wire up vqdmullh_laneq_s16 and vqdmullh_laneq_s32

2014-09-24 Thread James Greenhalgh
Hi, As per the subject line this patch adds support for two arm_neon.h intrinsics that we had missed. We also need to fix the signature of vqdmulls_lane_s32, which is an obvious extension to this patch while we are in the area. Tested for simd.exp and aarch64.exp with no issues. OK? Thanks,

Re: [PATCH] Fix PR 58867: asan and ubsan tests not run for installed testing.

2014-09-24 Thread Maxim Ostapenko
Hi Andrew! I tried to run ASan and UBSan tests on installed toolchain, but failed because current GCC doesn't support this opportunity. I see, you had fixed this issue (http://patchwork.ozlabs.org/patch/286866/), but the patch wasn't applied to GCC. So, I wonder if you are going to commit

Re: PING: Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-24 Thread Jeff Law
On 09/24/14 08:40, Aldy Hernandez wrote: On 09/18/14 12:39, Aldy Hernandez wrote: Yeah, sure, either way it's a good cleanup ;). No strong opinions. Though I think true/false are the way we want folks to write new code. Given that's the long term direction, might as well fix that nit for

Re: [PATCH, Pointer Bounds Checker 22/x] Inline

2014-09-24 Thread Jeff Law
On 09/24/14 01:28, Ilya Enkovich wrote: I'm a bit curious why you removed the original RETBND statement in value-prof, only to reinsert it. Is there some reason you needed to do that? After call transformation we have smth like that: if (confition) new_lhs = direct_call (...); else

Re: [GOOGLE] Fix new tests

2014-09-24 Thread Xinliang David Li
not sure if there is a better way, but ok. David On Wed, Sep 24, 2014 at 6:20 AM, Teresa Johnson tejohn...@google.com wrote: The new tests added for -mpatch-functions-for-instrumentation did not correctly restrict themselves to x86_64 since tree-prof.exp doesn't support dg-do. Work around

[Patch] Fix PR61889 for the w64-mingw32 case

2014-09-24 Thread Rainer Emrich
The following patch fixes PR61889 for x86_64-w64-mingw32. Details can be found on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 The patch was bootstrapped on x86_64-w64-mingw32. If patch the patch is ok, Kai would you apply, please? Rainer 2014-09-24 Rainer Emrich

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

2014-09-24 Thread Mike Stump
On Sep 24, 2014, at 5:54 AM, Jonathan Wakely jwak...@redhat.com 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?

[wwwdocs] IPA/LTO/FDO updates for gcc-5/changes.html

2014-09-24 Thread Jan Hubicka
Hi, this patch adds list of changes to IPA/LTO/FDO before I forget about them ;) Honza Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v retrieving revision 1.9 diff -c -p -r1.9 changes.html ***

Re: Enable EBX for x86 in 32bits PIC code

2014-09-24 Thread Jeff Law
On 09/24/14 00:56, Ilya Enkovich wrote: 2014-09-23 20:10 GMT+04:00 Jeff Law l...@redhat.com: 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

Re: [GOOGLE] Fix new tests

2014-09-24 Thread Teresa Johnson
On Wed, Sep 24, 2014 at 8:23 AM, Xinliang David Li davi...@google.com wrote: not sure if there is a better way, but ok. I looked through the documentation and other tests last night, but couldn't come up with a better way unfortunately. Teresa David On Wed, Sep 24, 2014 at 6:20 AM, Teresa

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Michael Eager
On 09/24/14 07:31, Chen Gang wrote: Hello Michael: Firstly, thank you very much for always providing your aid to me for microblaze. At present, after try testsuite, the result is much better than my original trying, please help check the result: is it enough for our microblaze testsuite (can

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Mike Stump
On Sep 24, 2014, at 8:28 AM, Michael Eager ea...@eagerm.com wrote: After check the current result log, I find many remote target test related sentences, do we have to process it? e.g. Download to microblaze-xilinx-gdb failed, couldn't execute rcp: no such file or directory. The test

Re: [PATCH i386 AVX512] [51/n] Add pd2dq and dq2pd converts.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 10:49 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, Patch in the bottom adds support for pd2dq and dq2pd conversions. 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, Pointer Bounds Checker 33/x] MPX ABI

2014-09-24 Thread Jeff Law
On 09/24/14 01:05, Ilya Enkovich wrote: However, we've still got the problem that the RTL you've generated is ill-formed. If I understand things correctly, the assignments are the result of the call, that should be modeled by having the destination be a PARALLEL as mentioned earlier. OK. Will

Re: parallel check output changes?

2014-09-24 Thread Segher Boessenkool
On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: On 09/23/2014 11:33 AM, Richard Sandiford wrote: Your patch instead sorts based on the full test name, including options, which means that the output no longer matches what you'd get from a non-parallel run. AFAICT, it also no

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Chen Gang
On 09/24/2014 11:28 PM, Michael Eager wrote: On 09/24/14 07:31, Chen Gang wrote: Hello Michael: Firstly, thank you very much for always providing your aid to me for microblaze. At present, after try testsuite, the result is much better than my original trying, please help check the result:

Re: [PATCH i386 AVX512] [52/n] Add convert ps2pd and ps2dq.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 10:54 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, Patch in the bottom adds support for ps2dq and ps2pd conversions. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md

[jit] Add copyright and license headers and footers

2014-09-24 Thread David Malcolm
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: [...] diff --git a/gcc/jit/internal-api.c b/gcc/jit/internal-api.c Should start with standard copyright and license header. This applies to all sources in gcc/jit/. [...] I've committed the following to the dmalcolm/jit branch:

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Chen Gang
On 09/24/2014 11:37 PM, Mike Stump wrote: On Sep 24, 2014, at 8:28 AM, Michael Eager ea...@eagerm.com wrote: After check the current result log, I find many remote target test related sentences, do we have to process it? e.g. Download to microblaze-xilinx-gdb failed, couldn't execute rcp:

Re: [PATCH i386 AVX512] [54/n] Add mov[dlh]dup insns support.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 2:51 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, patch in the bottom introduces support for vmov[dlh]dup insns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md

Re: parallel check output changes?

2014-09-24 Thread Andrew MacLeod
On 09/24/2014 12:10 PM, Segher Boessenkool wrote: On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: On 09/23/2014 11:33 AM, Richard Sandiford wrote: Your patch instead sorts based on the full test name, including options, which means that the output no longer matches what you'd

Re: [PATCH i386 AVX512] [53/n] Update vec_setmode_0 pattern constraints.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 2:48 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, Patch in the bottom extends to EVEX constraints of vec_setmode_0 insn pattern. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ *

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

2014-09-24 Thread Jonathan Wakely
On 23/09/14 21:58 +0200, François Dumont wrote: On 23/09/2014 13:22, Jonathan Wakely wrote: On 22/09/14 23:51 +0200, François Dumont wrote: New patch in a couple of day then. OK, thanks. It was faster than I though, here is the fixed patch tested under Linux x86_64. [snip] Ok to commit ?

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

2014-09-24 Thread Mark Wielaard
Hi Andreas, On Wed, 2014-09-24 at 14:40 +0200, Andreas Arnez wrote: I changed the patch a bit further, to reduce unnecessary iterations and recursions, and tested it again. Thanks for adding the tests and the testing. I think in general it is a nicer and cleaner fix than I did. I do have a

Re: [PATCH i386 AVX512] [55/n] Extend `perm' insn patterns.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 2:53 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, Patch in the bottom extends `perm' 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

Re: [PATCH 1/4] [AARCH64,NEON] Add patterns + builtins for vld[234](q?)_lane_* intrinsics

2014-09-24 Thread Charles Baylis
Kyril, Tejas, Thanks for the review. I agree with all points and will respin v2 accordingly Charles

[jit] Use standard initial includes

2014-09-24 Thread David Malcolm
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: [...] +#include config.h +#include system.h +#include ansidecl.h +#include coretypes.h The standard initial includes are config.h, system.h, coretypes.h. system.h includes libiberty.h which includes ansidecl.h, so direct

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Mike Stump
On Sep 24, 2014, at 9:23 AM, Chen Gang gang.chen.5...@gmail.com wrote: For one simple solving way under fedora: yum install rsh, and I will get another issue: Download to microblaze-xilinx-gdb failed, microblaze-xilinx-gdb: Unknown host So I guess the root cause is: I only use

  1   2   >