Re: [Patch, libstdc++/65420] Use constexpr variables as regex_constans flags

2015-03-15 Thread Tim Shen
On Sun, Mar 15, 2015 at 5:59 AM, Daniel Krügler wrote: (I switched to a less interesting but practical change, so all class stuff is gone. But I'd like to reply to your comments.) > Your implementation choice is an interesting approach. I believe that > a strict reading of the library specificat

[AArch64][PR65375] Fix RTX cost for vector SET

2015-03-15 Thread Kugan
AArch64 RTX cost for vector SET is causing PR65375. Lower subreg is using this rtx_cost to compute the cost of moves, and splitting anything larger than word size, 64-bits in this case. The aarch64 rtx_costs is returning 2 * COST_N_INSNS(1) for vector moves, so they get split. Attach patch fixes t

Re: [RS6000] bswapdi2 pattern, reload and lra

2015-03-15 Thread Alan Modra
On Sun, Mar 15, 2015 at 10:51:02PM -0400, David Edelsohn wrote: > On Tue, Dec 17, 2013 at 6:50 AM, Alan Modra wrote: PR target/61350 > > gcc/ > > * config/rs6000/rs6000.md (bswapdi2): Remove one scratch reg. > > Modify Z->r bswapdi splitter to use dest in place of scratch.

Re: PR c++/64626 - C++14 single quote should not always be a digit separator

2015-03-15 Thread Ed Smith-Rowland
OK, Here is a new try at PR c++/64626 - C++14 single quote should not always be a digit separator. I decided to look for multiple terminating single quotes. The test cases have bee adjusted and debugged. Built and tested on x86_64-linux. Ed libcpp/ 2015-03-16 Edward Smith-Rowland <3dw.

Re: [RS6000] bswapdi2 pattern, reload and lra

2015-03-15 Thread David Edelsohn
On Tue, Dec 17, 2013 at 6:50 AM, Alan Modra wrote: > This patch is aimed at fixing test failures introduced by my > 2013-12-07 change to bswapdi2_32bit: > FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times lwbrx 6 > FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times stwbrx 6 > > The 2013

RE: [PATCH][ARM] New testcase to check parameter passing bug

2015-03-15 Thread Honggyu Kim
Hi, I have modified and moved the testcase following your comments. (from gcc.target/arm to gcc.dg) Please let me know if there's still something to fix more. I appreciate all your comments. Honggyu --- gcc/testsuite/ChangeLog|4 gcc/testsuite/gcc.dg/pr65358.c | 33 +++

[committed] Fix gcc.dg/torture/pr65270-1.c an gcc.dg/torture/pr65270-2.c on hppa*-*-hpux*

2015-03-15 Thread John David Anglin
The attach change adds the -fno-common option on hppa*-*-hpux*. This provides necessary alignment. Tested on hppa2.0w-hp-hpux11.11. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2015-03-15 John David Anglin * gcc.dg/torture/pr65270-1.c: Add -fno-common

Re: [PATCH] pr 63354 - gcc -pg -mprofile-kernel creates unused stack frames on leaf functions on ppc64le

2015-03-15 Thread Martin Sebor
On 03/14/2015 08:34 AM, Segher Boessenkool wrote: On Fri, Mar 13, 2015 at 03:54:57PM -0600, Martin Sebor wrote: Attached is a patch that eliminates the unused stack frame allocated by gcc 5 with -pg -mprofile-kernel on powepc64le and brings the code into parity with previous gcc versions. The p

[Patch, fortran] PR59198 - [4.8/4.9/5 Regression] ICE on cyclically dependent polymorphic types

2015-03-15 Thread Paul Richard Thomas
Dear All, As will be apparent from the PR, I have spent a silly amount of time on this one :-( Once I became 'de-obsessed' with the fact that the reduced testcase worked, when 'rng' was made a pointer and concentrated on the procedure pointer component 'obs1_int', finding the problem was rather m

Re: [PATCH, PR target/65103, 2/3] Propagate address constants into loops for i386

2015-03-15 Thread Richard Sandiford
Ilya Enkovich writes: > This patch allows propagation of loop invariants for i386 if propagated > value is a constant to be used in address operand. Bootstrapped and > tested on x86_64-unknown-linux-gnu. OK for trunk or stage 1? Is it necessary for this to be a target hook? The concept doesn't

Re: [PATCH PR64820] Fix ASan UAR detection fails on 32-bit targets if SSP is enabled.

2015-03-15 Thread Richard Sandiford
Maxim Ostapenko writes: > @@ -293,17 +302,15 @@ alloc_stack_frame_space (HOST_WIDE_INT size, unsigned > HOST_WIDE_INT align) >new_frame_offset = frame_offset; Think this assignment is dead after your change. >if (FRAME_GROWS_DOWNWARD) > { > - new_frame_offset -= size + frame_p

Re: [Patch, libstdc++/65420] Use constexpr variables as regex_constans flags

2015-03-15 Thread Jonathan Wakely
On 15 March 2015 at 08:09, Tim Shen wrote: > Did a little bit refectoring on regex_constants, so that users won't > be polluted too much (they are not enum types anymore). I think this is overengineered and unnecessary. All it needs is something like: enum syntax_option_type : unsigned { }; cons

Re: [Patch, libstdc++/65420] Use constexpr variables as regex_constans flags

2015-03-15 Thread Daniel Krügler
2015-03-15 9:09 GMT+01:00 Tim Shen : > Did a little bit refectoring on regex_constants, so that users won't > be polluted too much (they are not enum types anymore). Your implementation choice is an interesting approach. I believe that a strict reading of the library specification does not allow t

[Patch, libstdc++/65420] Use constexpr variables as regex_constans flags

2015-03-15 Thread Tim Shen
Did a little bit refectoring on regex_constants, so that users won't be polluted too much (they are not enum types anymore). Bootstrapped and tested. Thanks! -- Regards, Tim Shen commit 5cd86b408ef0a9cba1a21a3018c797d9e245d158 Author: Tim Shen Date: Sat Mar 14 23:05:05 2015 -0700 P