Re: Fix PR52482, libitm compilation in OSX ppc with old cctools

2015-07-06 Thread Carlos Sánchez de La Lama
>> Patch is against gcc-4.8.4, but affected lines have not changed in SVN HEAD. > > I dropped this into all active release branches as well. Ok, thanks :) > If you do a test suite run, feel free to email it to the test results > list. I am running "make check" on my (locally patched) 4.8.4. Will

Re: [PATCH] PR target/53383: Allow -mincoming-stack-boundary=3 with -mno-sse

2015-07-06 Thread Uros Bizjak
On Sun, Jul 5, 2015 at 11:40 PM, H.J. Lu wrote: > Similar to -mpreferred-stack-boundary=3, -mincoming-stack-boundary=3 is > allowed with -mno-sse in 64-bit mode. > > OK for trunk? > > > H.J. > gcc/ > > PR target/53383 > * config/i386/i386.c (ix86_option_override_internal): Allow >

[PATCH][AArch64] PR target/66731 Fix fnmul insn with -frounding-math

2015-07-06 Thread Szabolcs Nagy
fnmul was modeled as (-a)*b instead of -(a*b), which is wrong with -frounding-math, so the correct pattern is added too and the other one is only used if !flag_rounding_math. This affects a glibc math test, similar fix will be needed for ARM. Tested with aarch64-none-linux-gnu cross compiler. is

Re: [PATCH, fortran] Fix numerous ICEs in IO statements with IOMSG

2015-07-06 Thread FX
> 2015-07-05 Steven G. Kargl > > * io.c (check_char_variable): New function. > (match_open_element, match_close_element, match_file_element, > match_dt_element, match_inquire_element, match_wait_element): Use it. > > > 2015-07-05 Steven G. Kargl > > * gfortran.dg/i

Re: [PATCH][8/n] Remove GENERIC stmt combining from SCCVN

2015-07-06 Thread Eric Botcazou
> Hum, somehow I convinced myself that it was ok if the precision > wasn't the same (but I can't remember my line of thought). Your > testcase clearly shows I was wrong ;) It's not mine, but Zhendong Su's. For the sake of completeness, I also had an Ada testcase, but I know how convincing C tes

Rename read-md.c:decimal_string

2015-07-06 Thread Rainer Orth
One of the recent header file changes (haven't check which) broke mainline Solaris bootstrap: /vol/gcc/src/hg/trunk/local/gcc/read-md.c: In function 'char* decimal_string(int)': /vol/gcc/src/hg/trunk/local/gcc/read-md.c:782:27: error: 'char* decimal_string(int)' redeclared as different kind of s

[RFC] two-phase marking in gt_cleare_cache

2015-07-06 Thread Tom de Vries
Hi, Using attached untested patch, I managed to minimize a test-case failure for PR 66714. The patch introduces two-phase marking in gt_cleare_cache: - first phase, it loops over all the hash table entries and removes those which are dead - second phase, it runs over all the live hash table

Re: Rename read-md.c:decimal_string

2015-07-06 Thread Richard Sandiford
Rainer Orth writes: > 2015-07-06 Rainer Orth > > * read-md.c (decimal_string): Rename to ... > (md_decimal_string): ... this. > (handle_enum): Reflect this. OK, thanks. Richard

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-06 Thread Alan Lawrence
Richard Biener wrote: I also believe this loop is equivalent to checking TYPE_ALIGN of the aggregate type? Jakub is correct: the intention is to discard any top-level alignment attribute on a struct declaration. I'll double check your wording in the abi document, but it seems to be unclea

[PING][PATCH, 1/2] Merge rewrite_virtuals_into_loop_closed_ssa from gomp4 branch

2015-07-06 Thread Tom de Vries
On 25/06/15 09:42, Tom de Vries wrote: Hi, this patch merges rewrite_virtuals_into_loop_closed_ssa (originally submitted here: https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01236.html ) to trunk. Bootstrapped and reg-tested on x86_64. OK for trunk? Ping. Thanks, - Tom 0001-Merge-rewrite

[PING][PATCH, 2/2][PR66642] Add empty loop exit block in transform_to_exit_first_loop_alt

2015-07-06 Thread Tom de Vries
On 25/06/15 09:43, Tom de Vries wrote: Hi, I ran into a failure with parloops for reduction loop testcase libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c. When we exercise the low iteration count loop, the test-case fails. To understand the problem, let's first look at what happen

Re: [PATCH 1/3] [ARM] PR63870 NEON error messages

2015-07-06 Thread Alan Lawrence
I note some parts of this duplicate my https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01422.html , which has been pinged a couple of times. Both Charles' patch, and my two, contain parts the other does not... Cheers, Alan Charles Baylis wrote: gcc/ChangeLog: Charles Baylis * con

Re: [PATCH][AArch64] PR target/66731 Fix fnmul insn with -frounding-math

2015-07-06 Thread James Greenhalgh
On Mon, Jul 06, 2015 at 09:20:39AM +0100, Szabolcs Nagy wrote: > fnmul was modeled as (-a)*b instead of -(a*b), which is wrong with > -frounding-math, so the correct pattern is added too and the other > one is only used if !flag_rounding_math. > > This affects a glibc math test, similar fix will b

Re: [commited, Patch, Fortran, PR58586, v5] ICE with derived type with allocatable component passed by value

2015-07-06 Thread Andre Vehreschild
Hi Steve, hi Paul, hi all, Steve and Paul, thank you very much for the reviews. Committed with the requested changes as r225447 and r225448. The last commit adds the Changelog entry in the testsuite I forgot. Sorry for that. For the open issue in the testcase I have opened the pr: https://gcc.gn

[PATCH] Fix PR66759

2015-07-06 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-07-06 Richard Biener PR middle-end/66759 * match.pd: Add missing constraint of y to REAL_CST in REAL_CST - x CMP y to y - CST CMP x simplification. * gcc.dg/torture/pr66759.c: New tes

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-06 Thread Alan Lawrence
Eric Botcazou wrote: Technically this is incorrect since AGGREGATE_TYPE_P includes ARRAY_TYPE and ARRAY_TYPE doesn't have TYPE_FIELDS. I doubt we could reach that case though (unless there's a language that allows passing arrays by value). Ada passes small array types by the method specified b

Re: [Patch, Fortran, 66035, v2] [5/6 Regression] gfortran ICE segfault

2015-07-06 Thread Andre Vehreschild
Hi all, please find attached the next version of the patch for pr66035 fixing an ICE. Scope (copied from first submit): An ICE occurred when in a structure constructor an allocatable component of type class was initialized with an existing class object. This was caused by - the size of the memo

[PATCH 0/7] more ifdef removal

2015-07-06 Thread tbsaunde+gcc
From: Trevor Saunders Hi, $subject. patches individually bootstrapped + regtested on x86_64-linux-gnu, and the series was run through config-list.mk with some other stuff a couple weeks ago. I plan to commit this as preapproved tonight if nobody complains. Trev Trevor Saunders (7): reduce

[PATCH 2/7] remove #if for HAVE_cc0 in combine.c

2015-07-06 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-07-06 Trevor Saunders * combine.c (do_SUBST_MODE): Don't check the value of HAVE_cc0 with the preprocessor. (combine_instructions): Likewise. (try_combine): Likewise. (subst): Likewise. (distribute_note

[PATCH 1/7] reduce conditional compilation for LOAD_EXTEND_OP

2015-07-06 Thread tbsaunde+gcc
From: Trevor Saunders Provide a default in files where that is possible, so that everything else there can be unconditionally compiled. However rtlanal.c and reload.c do tricky things that break providing a global default, so we can't do that yet. gcc/ChangeLog: 2015-07-06 Trevor Saunders

[PATCH 7/7] always define WORD_REGISTER_OPERATIONS

2015-07-06 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-07-06 Trevor Saunders * defaults.h: Provide default for WORD_REGISTER_OPERATIONS. * config/alpha/alpha.h: Define WORD_REGISTER_OPERATIONS to 1. * config/arc/arc.h: Likewise. * config/arm/arm.h: Likewise. * conf

[PATCH 3/7] always define SHORT_IMMEDIATES_SIGN_EXTEND

2015-07-06 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-07-06 Trevor Saunders * combine.c (update_rsp_from_reg_equal): Don't check if SHORT_IMMEDIATES_SIGN_EXTEND is defined. (reg_nonzero_bits_for_combine): Likewise. * config/alpha/alpha.h: Define SHORT_IMMEDIATES_SIGN_EXTE

[PATCH 5/7] always define AUTO_INC_DEC

2015-07-06 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-07-06 Trevor Saunders * rtl.h: Always define AUTO_INC_DEC. * auto-inc-dec.c (pass_inc_dec::execute): Adjust. * combine.c (combine_instructions): Likewise. (can_combine_p): Likewise. (try_combine): Likewise.

[PATCH 6/7] reduce conditional compilation based on AUTO_INC_DEC

2015-07-06 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-07-06 Trevor Saunders * auto-inc-dec.c (pass_inc_dec::execute): Don't check the value of AUTO_INC_DEC with the preprocessor. * combine.c (combine_instructions): Likewise. (can_combine_p): Likewise. (try_combine

[PATCH 4/7] use #if for HARD_FRAME_POINTER_IS_FRAME_POINTER less

2015-07-06 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-07-06 Trevor Saunders * combine.c (can_combine_def_p): Don't check the value of * HARD_FRAME_POINTER_IS_FRAME_POINTER with the preprocessor. (combinable_i3pat): Likewise. (mark_used_regs_combine): Likewise. * r

[PATCH] Fix PR66772

2015-07-06 Thread Richard Biener
In this PR we hit the issue that when CCP faces a conditional which is (still) undefined it will consider all outgoing edges executable. Eventually we'll reverse that decision (but without reflecting that in the lattice), but it's too late then if a PHI node merging the edges has been evaluated to

RE: [PATCH] MIPS: fix failing branch range checks for micromips

2015-07-06 Thread Andrew Bennett
> There is a follow-up patch that I will be working on that will correctly > update the other > branch tests to correctly test out of range branch behaviour for > micromips.  Currently these > are passing because the mips branch range offset is large enough.  These > offsets will > need to be reduc

[PATCH] Make SSA propagator iteration order consistent

2015-07-06 Thread Richard Biener
The intent (as I read it) of the iteration order in ssa_propagate is to process stmts in the following order: 1) complete simulation of BBs from making one of their entries executable 2) simulation of stmts fed by stmts that changed to VARYING 3) simulation of the rest of stmts fed by stmts th

Re: [PATCH] config/bfin/bfin.c (hwloop_optimize): Use return false instead of gcc_assert for checking jump_insn.

2015-07-06 Thread Bernd Schmidt
On 07/03/2015 04:13 AM, Chen Gang wrote: On 07/01/2015 11:27 PM, Chen Gang wrote: On 7/1/15 21:52, Bernd Schmidt wrote: Below is a patch. Can you test this with anything you have beyond the testsuite? It can fix this issue (Bug66620), let the insns standard, and can build the bfin kernel wit

[RFC 2/2] Add steady_clock support to condition_variable

2015-07-06 Thread Mike Crowe
If __gthread_cond_timedwaitonclock is available it can be used it to fix part of https://gcc.gnu.org/bugzilla/show_bug.cgi?id=41861 by supporting std::chrono::steady_clock properly with std::condition_variable. This means that code using std::condition_variable::wait_for or std::condition_variable

[Patch, fortran, pr66578, v1] [F2008] Invalid free on allocate(...,source=a(:)) in block

2015-07-06 Thread Andre Vehreschild
Hi all, this is a proposal to patch PR 66578 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66578 . It extends work of Mikael Morin. The patch fixes two issues: 1. a source'd allocate in a block: allocate(c, source=a(:)). The issues occurs because due to the new handling of source-expressions in tr

ORDER

2015-07-06 Thread Anthony Lackner Stores .
Hello, I am Anthony Lackner of Anthony Lackner store.I want to order from your company and I will like to know if you accept credit payment and the types of card accepted by you. About the shipment I have a freight forwarder (http://www.popular-shipping.com) that will handle the shipment when

[Patch,microblaze]: Optimized usage of reserved stack space for function arguments.

2015-07-06 Thread Ajit Kumar Agarwal
All: The below patch optimized the usage of the reserved stack space for function arguments. The stack space is reserved if the function is a libcall, variable number of arguments, aggregate data types, and some parameter are reserved in registers and some parameters is reserved in the stack.

[PATCH] Fix PR66767

2015-07-06 Thread Richard Biener
Similar to the vect_gen_niters_for_prolog_loop change I already noticed on x86_64. Built on ppc64-unknown-linux-gnu, applied. Richard. 2015-07-06 Richard Biener PR tree-optimization/66767 * tree-vect-loop-manip.c (vect_create_cond_for_align_checks): Make sure to buil

[Ada] Add DragonFly support to System.OS_Constants template

2015-07-06 Thread John Marino
The System.OS_Constants templates for GNAT has three preprocessor checks for FreeBSD. In all three cases, DragonFly BSD needs to be treated the same as FreeBSD. The attached patch accomplishes this. Please consider incorporating the patch into trunk. Regards, John Index: gcc/ada/s-oscons-tmplt.

[PATCH] MIPS: For micromips allow near-far-3.c test to use the jals instruction to call near_func

2015-07-06 Thread Andrew Bennett
Hi, The near-far-3.c test is failing for micromips because it is expecting the call to near_func to be performed by a jal instruction, but for micromips this is done by a jals instruction. I have updated the expected test output to deal with this case. I have tested this on the mips-mti-elf

RE: [PATCH] MIPS: For micromips allow near-far-3.c test to use the jals instruction to call near_func

2015-07-06 Thread Moore, Catherine
> -Original Message- > From: Andrew Bennett [mailto:andrew.benn...@imgtec.com] > Sent: Monday, July 06, 2015 9:20 AM > To: gcc-patches@gcc.gnu.org > Cc: Matthew Fortune; Moore, Catherine > Subject: [PATCH] MIPS: For micromips allow near-far-3.c test to use the jals > instruction to call n

Re: [RFC] two-phase marking in gt_cleare_cache

2015-07-06 Thread Richard Biener
On Mon, Jul 6, 2015 at 10:57 AM, Tom de Vries wrote: > Hi, > > Using attached untested patch, I managed to minimize a test-case failure for > PR 66714. > > The patch introduces two-phase marking in gt_cleare_cache: > - first phase, it loops over all the hash table entries and removes > those whi

[PATCH] PR target/66749: Add -march=iamcu to optimize for IA MCU

2015-07-06 Thread H.J. Lu
IA MCU is based on Intel Pentium ISA without x87 and passing parameters in registers. We want to optimize for IA MCU without changing existing Pentium codegen. This patch adds PROCESSOR_IAMCU for -march=iamcu, which is based on -march=pentium with updated cost tables. OK for trunk? Thanks. H.

Re: [RFC] two-phase marking in gt_cleare_cache

2015-07-06 Thread Richard Biener
On Mon, Jul 6, 2015 at 3:25 PM, Richard Biener wrote: > On Mon, Jul 6, 2015 at 10:57 AM, Tom de Vries wrote: >> Hi, >> >> Using attached untested patch, I managed to minimize a test-case failure for >> PR 66714. >> >> The patch introduces two-phase marking in gt_cleare_cache: >> - first phase, it

[PATCH] MIPS: Do not generate micromips code for the no-smartmips-lwxs.c testcase

2015-07-06 Thread Andrew Bennett
Hi, The LWXS instruction is part of the micromips ISA which means it is valid to generate it for the no-smartmips-lwxs.c testcase. I have updated the dg-options for the test to ensure that it does not generate micromips code. I have tested this on the mips-mti-elf target using mips32r2/{-mno-m

Re: [Patch,microblaze]: Optimized usage of reserved stack space for function arguments.

2015-07-06 Thread Oleg Endo
Hi, Just some general comments... On 06 Jul 2015, at 22:05, Ajit Kumar Agarwal wrote: > +static bool > +microblaze_parm_needs_stack (cumulative_args_t args_so_far, tree type) > +{ > + enum machine_mode mode; 'enum' is not required in C++, please omit it. We've been trying

Re: flatten cfgloop.h

2015-07-06 Thread Michael Matz
Hi, On Sun, 5 Jul 2015, Prathamesh Kulkarni wrote: > Hi, > The attached patches flatten cfgloop.h. > patch-1.diff moves around prototypes and structures to respective > header-files. > patch-2.diff (mostly auto-generated) replicates cfgloop.h includes in c files. > Bootstrapped and tested on x86

[PATCH] Simplify vector compare-not-select sequence

2015-07-06 Thread Bill Schmidt
Hi, Due to specifics of the POWER architecture, some forms of a vector compare followed by a vector select are represented in RTL as a compare, followed by a logical NOT, followed by the select. This tends to end up generating an extra instruction. This patch adds a case to simplify-rtx.c to rem

Re: [PING][PATCH, 1/2] Merge rewrite_virtuals_into_loop_closed_ssa from gomp4 branch

2015-07-06 Thread Richard Biener
On Mon, 6 Jul 2015, Tom de Vries wrote: > On 25/06/15 09:42, Tom de Vries wrote: > > Hi, > > > > this patch merges rewrite_virtuals_into_loop_closed_ssa (originally > > submitted here: https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01236.html > > ) to trunk. > > > > Bootstrapped and reg-tested on

Re: [PING][PATCH, 2/2][PR66642] Add empty loop exit block in transform_to_exit_first_loop_alt

2015-07-06 Thread Richard Biener
On Mon, 6 Jul 2015, Tom de Vries wrote: > On 25/06/15 09:43, Tom de Vries wrote: > > Hi, > > > > I ran into a failure with parloops for reduction loop testcase > > libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c. When we > > exercise the low iteration count loop, the test-case fails

Re: [RFC 1/2] gthread: Add __gthread_cond_timedwaitonclock

2015-07-06 Thread Jonathan Wakely
On 06/07/15 13:55 +0100, Mike Crowe wrote: diff --git a/libgcc/gthr-posix.h b/libgcc/gthr-posix.h index fb59816..0e01866 100644 --- a/libgcc/gthr-posix.h +++ b/libgcc/gthr-posix.h @@ -33,6 +33,7 @@ see the files COPYING3 and COPYING.RUNTIME respectively. If not, see #define __GTHREADS_CXX0X 1

Re: [RFC 2/2] Add steady_clock support to condition_variable

2015-07-06 Thread Jonathan Wakely
On 06/07/15 13:55 +0100, Mike Crowe wrote: If __gthread_cond_timedwaitonclock is available it can be used it to fix part of https://gcc.gnu.org/bugzilla/show_bug.cgi?id=41861 by supporting std::chrono::steady_clock properly with std::condition_variable. This means that code using std::condition_

[PATCH] libstdc++ os_defines now required for DragonFly

2015-07-06 Thread John Marino
On the development branch of DragonFly BSD, it was discovered that __LONG_LONG_SUPPORTED was accidently unconditionally defined. This had a positive side effect of allowing GCC conftests to pass for C99 support via wchar.h. When the bug was fixed, the wchar C99 conftest now fails, resulting in a

Re: [PATCH 7/7] always define WORD_REGISTER_OPERATIONS

2015-07-06 Thread Segher Boessenkool
Hi Trevor, On Mon, Jul 06, 2015 at 08:11:30AM -0400, tbsaunde+...@tbsaunde.org wrote: > * defaults.h: Provide default for WORD_REGISTER_OPERATIONS. > * config/alpha/alpha.h: Define WORD_REGISTER_OPERATIONS to 1. > * config/arc/arc.h: Likewise. > * config/arm/arm.h: Likewise

Re: Clean-ups in match.pd

2015-07-06 Thread Richard Biener
On Sat, Jul 4, 2015 at 4:34 PM, Marc Glisse wrote: > Hello, > > these are just some minor changes. I believe I had already promised a build_ > function to match integer_each_onep. > > Bootstrap+testsuite on powerpc64le-unknown-linux-gnu (it looks like > *-match.c takes about 10 minutes to compile

Re: [Ada] Add DragonFly support to System.OS_Constants template

2015-07-06 Thread Thomas Quinot
* John Marino, 2015-07-06 : > The System.OS_Constants templates for GNAT has three preprocessor checks > for FreeBSD. In all three cases, DragonFly BSD needs to be treated the > same as FreeBSD. The attached patch accomplishes this. Thanks John, looks good to me! Thomas.

Re: [RFC 1/2] gthread: Add __gthread_cond_timedwaitonclock

2015-07-06 Thread Mike Crowe
On Monday 06 July 2015 at 14:51:42 +0100, Jonathan Wakely wrote: > On 06/07/15 13:55 +0100, Mike Crowe wrote: > >diff --git a/libgcc/gthr-posix.h b/libgcc/gthr-posix.h > >index fb59816..0e01866 100644 > >--- a/libgcc/gthr-posix.h > >+++ b/libgcc/gthr-posix.h > >@@ -33,6 +33,7 @@ see the files COPYI

RE: [PATCH] MIPS: For micromips allow near-far-3.c test to use the jals instruction to call near_func

2015-07-06 Thread Andrew Bennett
> OK. Committed as SVN 225457. Regards, Andrew

Re: [RFC 2/2] Add steady_clock support to condition_variable

2015-07-06 Thread Mike Crowe
On Monday 06 July 2015 at 14:54:06 +0100, Jonathan Wakely wrote: > Do you have a copyright assignment in place for GCC contributions? Not yet. From my reading of https://gcc.gnu.org/contribute.html I should probably ask you for the forms. :-) Mike.

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-06 Thread Ramana Radhakrishnan
On 06/07/15 12:00, Alan Lawrence wrote: > Eric Botcazou wrote: >>> Technically this is incorrect since AGGREGATE_TYPE_P includes ARRAY_TYPE >>> and ARRAY_TYPE doesn't have TYPE_FIELDS. I doubt we could reach that >>> case though (unless there's a language that allows passing arrays by value). >>

Re: [PATCH][10/n] Remove GENERIC stmt combining from SCCVN

2015-07-06 Thread Kyrill Tkachov
Hi Richard, On 01/07/15 14:03, Richard Biener wrote: This merges the complete comparison patterns from the match-and-simplify branch, leaving incomplete implementations of fold-const.c code alone. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-07-01 Richard Biene

[PATCH] MIPS: Update stack-1.c testcase to match micromips jraddiusp instruction.

2015-07-06 Thread Andrew Bennett
Hi, The stack-1.c testcase fails when being compiled for micromips with the -O0 optimization level. The reason is the testcase is expecting the following sequence at the end of the function: addiu $sp,$sp,16 jrc $31 But for micromips it generates the following: jradd

Re: [RFC 1/2] gthread: Add __gthread_cond_timedwaitonclock

2015-07-06 Thread Jonathan Wakely
On 06/07/15 15:10 +0100, Mike Crowe wrote: On Monday 06 July 2015 at 14:51:42 +0100, Jonathan Wakely wrote: This isn't correct, because it's possible to include before including any C++ Standard Library header, so the _GLIBCXX_ macro defined in libstdc++'s c++config.h will not have been defined

Re: [PATCH][10/n] Remove GENERIC stmt combining from SCCVN

2015-07-06 Thread Andreas Schwab
Kyrill Tkachov writes: > This broke some tests on aarch64: > FAIL: gcc.target/aarch64/subs.c scan-assembler subs\tw[0-9] > FAIL: gcc.target/aarch64/subs1.c scan-assembler subs\tw[0-9]+, w[0-9]+, > w[0-9]+ > FAIL: gcc.target/aarch64/subs1.c scan-assembler subs\tw[0-9]+, w[0-9]+, > w[0-9]+, lsl 3

Re: [PATCH 2/7] remove #if for HAVE_cc0 in combine.c

2015-07-06 Thread Segher Boessenkool
On Mon, Jul 06, 2015 at 08:11:25AM -0400, tbsaunde+...@tbsaunde.org wrote: > @@ -1327,7 +1322,7 @@ combine_instructions (rtx_insn *f, unsigned int nregs) >We need this special code because data flow connections >via CC0 do not get entered in LOG_LINKS. */ > > - if (

Re: [RFC 2/2] Add steady_clock support to condition_variable

2015-07-06 Thread Jonathan Wakely
On 06/07/15 15:18 +0100, Mike Crowe wrote: On Monday 06 July 2015 at 14:54:06 +0100, Jonathan Wakely wrote: Do you have a copyright assignment in place for GCC contributions? Not yet. From my reading of https://gcc.gnu.org/contribute.html I should probably ask you for the forms. :-) Yep :-)

Re: [PATCH 4/7] use #if for HARD_FRAME_POINTER_IS_FRAME_POINTER less

2015-07-06 Thread Segher Boessenkool
On Mon, Jul 06, 2015 at 08:11:27AM -0400, tbsaunde+...@tbsaunde.org wrote: > From: Trevor Saunders > > gcc/ChangeLog: > > 2015-07-06 Trevor Saunders > > * combine.c (can_combine_def_p): Don't check the value of > * HARD_FRAME_POINTER_IS_FRAME_POINTER with the preprocessor.

Re: [PATCH][10/n] Remove GENERIC stmt combining from SCCVN

2015-07-06 Thread Richard Biener
On Mon, 6 Jul 2015, Kyrill Tkachov wrote: > Hi Richard, > > On 01/07/15 14:03, Richard Biener wrote: > > This merges the complete comparison patterns from the match-and-simplify > > branch, leaving incomplete implementations of fold-const.c code alone. > > > > Bootstrapped and tested on x86_64-u

Re: [PATCH] libstdc++ os_defines now required for DragonFly

2015-07-06 Thread Jonathan Wakely
On 06/07/15 15:57 +0200, John Marino wrote: On the development branch of DragonFly BSD, it was discovered that __LONG_LONG_SUPPORTED was accidently unconditionally defined. This had a positive side effect of allowing GCC conftests to pass for C99 support via wchar.h. When the bug was fixed, the

Re: [PATCH][10/n] Remove GENERIC stmt combining from SCCVN

2015-07-06 Thread Kyrill Tkachov
On 06/07/15 15:46, Richard Biener wrote: On Mon, 6 Jul 2015, Kyrill Tkachov wrote: Hi Richard, On 01/07/15 14:03, Richard Biener wrote: This merges the complete comparison patterns from the match-and-simplify branch, leaving incomplete implementations of fold-const.c code alone. Bootstrappe

Re: Get rid of move_iterator in debug checks

2015-07-06 Thread Jonathan Wakely
On 05/07/15 23:23 +0200, François Dumont wrote: Ok ? Yes, OK, thanks.

RE: [PATCH] MIPS: Update stack-1.c testcase to match micromips jraddiusp instruction.

2015-07-06 Thread Matthew Fortune
Andrew Bennett writes: > The stack-1.c testcase fails when being compiled for micromips with the > -O0 optimization level. The reason is the testcase is expecting the > following sequence at the end of the function: > >addiu $sp,$sp,16 >jrc $31 > > But for micromips it gen

Re: [PATCH] fix PR46029: reimplement if conversion of loads and stores [2nd submitted version of patch]

2015-07-06 Thread Alan Lawrence
Abe wrote: On 7/2/15 4:49 AM, Alan Lawrence wrote: As before, I'm still confused here. This still returns false, i.e. bails out of if-conversion, if the statement could trap. Doesn't the scratchpad let us handle that? Or do we just not care because it won't be vectorizable anyway??? This seems

Re: [PATCH] PR target/66749: Add -march=iamcu to optimize for IA MCU

2015-07-06 Thread Uros Bizjak
On Mon, Jul 6, 2015 at 3:28 PM, H.J. Lu wrote: > IA MCU is based on Intel Pentium ISA without x87 and passing parameters > in registers. We want to optimize for IA MCU without changing existing > Pentium codegen. This patch adds PROCESSOR_IAMCU for -march=iamcu, > which is based on -march=pentiu

Re: [Patch, ARM]: remove TARGET_ASM_FILE_START_APP_OFF

2015-07-06 Thread Ramana Radhakrishnan
On 30/06/15 13:07, Christian Bruel wrote: > Hi, > > A little bit of polishing around arm/thumb attribute_target emission and > testing: Since the arch mode is emitted for each function, the file setting > becomes useless or redundant. > > for example with attr_thumb.c: > > =>.arm > =>

Re: [PATCH] PR target/66749: Add -march=iamcu to optimize for IA MCU

2015-07-06 Thread H.J. Lu
On Mon, Jul 6, 2015 at 8:13 AM, Uros Bizjak wrote: > On Mon, Jul 6, 2015 at 3:28 PM, H.J. Lu wrote: >> IA MCU is based on Intel Pentium ISA without x87 and passing parameters >> in registers. We want to optimize for IA MCU without changing existing >> Pentium codegen. This patch adds PROCESSOR_

Re: flatten cfgloop.h

2015-07-06 Thread Andrew MacLeod
On 07/06/2015 09:38 AM, Michael Matz wrote: Hi, On Sun, 5 Jul 2015, Prathamesh Kulkarni wrote: Hi, The attached patches flatten cfgloop.h. patch-1.diff moves around prototypes and structures to respective header-files. patch-2.diff (mostly auto-generated) replicates cfgloop.h includes in c fil

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-06 Thread Ilya Verbin
On Thu, Jul 02, 2015 at 00:06:58 +0300, Ilya Verbin wrote: > On Tue, Jun 30, 2015 at 18:10:44 +0200, Jakub Jelinek wrote: > > The thing is whether it is actually a good idea to allocate the enter data > > allocated objects together. > > In OpenMP 4.0, generally objects would be allocated and deallo

RE: [PATCH] MIPS: Update stack-1.c testcase to match micromips jraddiusp instruction.

2015-07-06 Thread Moore, Catherine
> -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Monday, July 06, 2015 11:00 AM > To: Andrew Bennett; gcc-patches@gcc.gnu.org > Cc: Moore, Catherine > Subject: RE: [PATCH] MIPS: Update stack-1.c testcase to match micromips > jraddiusp instruction. >

Re: [PATCH][AArch64] PR target/66731 Fix fnmul insn with -frounding-math

2015-07-06 Thread Marcus Shawcroft
On 6 July 2015 at 09:20, Szabolcs Nagy wrote: > 2015-07-06 Szabolcs Nagy > > * gcc.target/aarch64/fnmul-1.c: New. > * gcc.target/aarch64/fnmul-2.c: New. > * gcc.target/aarch64/fnmul-3.c: New. > * gcc.target/aarch64/fnmul-4.c: New. +float +foo_s (float a, float

Re: [ARM] Fix PR middle-end/65958

2015-07-06 Thread Ramana Radhakrishnan
On 18/06/15 20:02, Eric Botcazou wrote: >> Please mark this pattern with (set_attr "type" "multiple"). > > Done. > >> While I suspect that stack probing is done before any insns with invalid >> constants in the function, it would be better to model the length of >> this insn so that the minipoo

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-06 Thread Alan Lawrence
Trying to push these now (svn!), patch 2 is going first. I realize my second iteration of patch 1/2, dropped the testcases from the first version. Okay to include those as per https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00215.html ? Cheers, Alan Ramana Radhakrishnan wrote: On 06/07/15 12

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-06 Thread Ramana Radhakrishnan
On 06/07/15 17:38, Alan Lawrence wrote: > Trying to push these now (svn!), patch 2 is going first. > > I realize my second iteration of patch 1/2, dropped the testcases from the > first version. Okay to include those as per > https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00215.html ? Yeah the

[PATCH] Optimize i?86-*-elfiamcu for iamcu by default

2015-07-06 Thread H.J. Lu
Default -mtune=/-march= to iamcu for i[34567]86-*-elfiamcu targets. OK for trunk? Thanks. H.J. --- * config.gcc (x86_archs): Add iamcu. (with_cpu): Default to iamcu for i[34567]86-*-elfiamcu. (with_arch): Likewise. * doc/invoke.texi: Add iamcu. --- gcc/config.gcc

Re: [PATCH 1/2][ARM] PR/65956 AAPCS update for alignment attribute

2015-07-06 Thread Alan Lawrence
Ramana Radhakrishnan wrote: On 06/07/15 17:38, Alan Lawrence wrote: Trying to push these now (svn!), patch 2 is going first. I realize my second iteration of patch 1/2, dropped the testcases from the first version. Okay to include those as per https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00

[PATCH, i386]: Use extv and extzv

2015-07-06 Thread Uros Bizjak
Hello! Attached patch renames (obsolete) extv and extzv expanders to extv and extzv. Since the patch uses SImode for operands 2 and 3 on both, 32bit and 64bit targets, we can remove a couple of BT patterns that were used for DImode operands. The patch also removes some modes from const_int operan

Re: [PATCH] Optimize i?86-*-elfiamcu for iamcu by default

2015-07-06 Thread Uros Bizjak
On Mon, Jul 6, 2015 at 6:43 PM, H.J. Lu wrote: > Default -mtune=/-march= to iamcu for i[34567]86-*-elfiamcu targets. > > OK for trunk? > > Thanks. > > H.J. > --- > * config.gcc (x86_archs): Add iamcu. > (with_cpu): Default to iamcu for i[34567]86-*-elfiamcu. > (with_arch):

Re: [PATCH] PR target/66749: Add -march=iamcu to optimize for IA MCU

2015-07-06 Thread Dominique d'Humières
This breaks bootstrap on x86_64-apple-darwin14: ../../work/gcc/config/i386/i386-c.c: In function 'void ix86_target_macros_internal(long long int, processor_type, processor_type, fpmath_unit, void (*)(cpp_reader*, const char*))':../../work/gcc/config/i386/i386-c.c:59:10: error: enumeration value

Re: [patch] [fixincludes] Ignore .DS_Store junk files when running make check

2015-07-06 Thread Andreas Schwab
Eric Gallager writes: > I was just matching the code that was already used there... should the > lines to ignore the CVS and .svn folders be re-written into the style > you propose, too? Yes, that would be an improvement. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint =

Re: [PATCH] PR target/66749: Add -march=iamcu to optimize for IA MCU

2015-07-06 Thread H.J. Lu
On Mon, Jul 6, 2015 at 9:58 AM, Dominique d'Humières wrote: > This breaks bootstrap on x86_64-apple-darwin14: > > ../../work/gcc/config/i386/i386-c.c: In function 'void > ix86_target_macros_internal(long long int, processor_type, processor_type, > fpmath_unit, void (*)(cpp_reader*, const > char

Re: [PATCH] Enable two UNSIGNED_FLOAT simplifications in simplify_unary_operation_1

2015-07-06 Thread Kyrill Tkachov
On 23/06/15 11:43, Renlin Li wrote: Hi Christophe, Yes, we have also noticed this failure. Here I have a simple patch to remove the mfloat-abi option for hard-float toolchain. The default abi is used. For non-hardfloat toolchain, softfp abi is specified. I have checked with arm-none-eabi and

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-06 Thread Jakub Jelinek
On Mon, Jul 06, 2015 at 06:34:25PM +0300, Ilya Verbin wrote: > On Thu, Jul 02, 2015 at 00:06:58 +0300, Ilya Verbin wrote: > > On Tue, Jun 30, 2015 at 18:10:44 +0200, Jakub Jelinek wrote: > > > The thing is whether it is actually a good idea to allocate the enter data > > > allocated objects togethe

Re: [RFC] two-phase marking in gt_cleare_cache

2015-07-06 Thread Tom de Vries
On 06/07/15 15:29, Richard Biener wrote: On Mon, Jul 6, 2015 at 3:25 PM, Richard Biener wrote: On Mon, Jul 6, 2015 at 10:57 AM, Tom de Vries wrote: Hi, Using attached untested patch, I managed to minimize a test-case failure for PR 66714. The patch introduces two-phase marking in gt_cleare_

RE: [Patch, MIPS] Enable fp-contract on MIPS and update -mfused-madd

2015-07-06 Thread Steve Ellcey
On Mon, 2015-06-29 at 12:07 -0700, Matthew Fortune wrote: > If nobody can identify any further functional issues with this patch then > I'd like to get this committed and pursue enhancements as a second round. > Catherine, would you be happy for this to be committed on that basis? > > Thanks, > M

Re: [PATCH 2/2][ARM] fix movdi expander to avoid illegal ldrd/strd

2015-07-06 Thread Alan Lawrence
Richard Earnshaw wrote: On 03/07/15 16:27, Alan Lawrence wrote: The previous patch caused a regression in gcc.c-torture/execute/20040709-1.c at -O0 (only), and the new align_rec2.c test fails, both outputting an illegal assembler instruction (ldrd on an odd-numbered reg) from output_move_double

Re: [PATCH] fix PR46029: reimplement if conversion of loads and stores [2nd submitted version of patch]

2015-07-06 Thread Abe
[Abe wrote:] This seems like an opportunity for more optimization in the future [On 7/6/15 10:09 AM, Alan Lawrence wrote:] we get enough benefit from the patch, even without my suggested extra change. Ok, fair enough! Thanks for the clarification. You are welcome, sir. [Alan wrote:]

Re: [PATCH] PR target/66749: Add -march=iamcu to optimize for IA MCU

2015-07-06 Thread Uros Bizjak
On Mon, Jul 6, 2015 at 7:03 PM, H.J. Lu wrote: > On Mon, Jul 6, 2015 at 9:58 AM, Dominique d'Humières > wrote: >> This breaks bootstrap on x86_64-apple-darwin14: >> >> ../../work/gcc/config/i386/i386-c.c: In function 'void >> ix86_target_macros_internal(long long int, processor_type, processor_

Re: flatten cfgloop.h

2015-07-06 Thread Jeff Law
On 07/06/2015 07:38 AM, Michael Matz wrote: Hi, On Sun, 5 Jul 2015, Prathamesh Kulkarni wrote: Hi, The attached patches flatten cfgloop.h. patch-1.diff moves around prototypes and structures to respective header-files. patch-2.diff (mostly auto-generated) replicates cfgloop.h includes in c fil

[gomp4.1] Parsing of defaultmap(tofrom:scalar) and private/firstprivate clauses on target construct

2015-07-06 Thread Jakub Jelinek
Hi! This patch adds parsing of defaultmap(tofrom:scalar) clause as well as allowing of private/firstprivate on target construct. Next step will be to handle private/firstprivate in clause splitting and gimplification rules, then expansion of firstprivate and private with pointer-assignment on tar

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-06 Thread Ilya Verbin
On Mon, Jul 06, 2015 at 19:25:09 +0200, Jakub Jelinek wrote: > On Mon, Jul 06, 2015 at 06:34:25PM +0300, Ilya Verbin wrote: > > On Thu, Jul 02, 2015 at 00:06:58 +0300, Ilya Verbin wrote: > > > The patch is not ready though, I don't know how to unmap GOMP_MAP_POINTER > > > vars. > > > In gomp_unmap

[PATCH 0/7] Fix bugs found during demangler fuzz-testing

2015-07-06 Thread Mikhail Maltsev
Hi, all! I performed some fuzz-testing of C++ demangler (libiberty/cp-demangle.c). The test revealed several bugs, which are addressed by this series of patches. Here is a brief description of them: First one adds a macro CHECK_DEMANGLER. When this macro is defined, d_peek_next_char and d_advance

[PATCH 1/7] Add CHECK_DEMANGLER

2015-07-06 Thread Mikhail Maltsev
--- libiberty/cp-demangle.h | 33 +++-- 1 file changed, 31 insertions(+), 2 deletions(-) diff --git a/libiberty/cp-demangle.h b/libiberty/cp-demangle.h index 6fce025..c37a91f 100644 --- a/libiberty/cp-demangle.h +++ b/libiberty/cp-demangle.h @@ -135,12 +135,41 @@ struc

[PATCH 2/7] Fix build with CP_DEMANGLE_DEBUG

2015-07-06 Thread Mikhail Maltsev
--- libiberty/cp-demangle.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libiberty/cp-demangle.c b/libiberty/cp-demangle.c index 2988b6b..12093cc 100644 --- a/libiberty/cp-demangle.c +++ b/libiberty/cp-demangle.c @@ -715,7 +715,7 @@ d_dump (struct demangle_component *dc, int

[PATCH 3/7] Fix trinary op

2015-07-06 Thread Mikhail Maltsev
--- libiberty/cp-demangle.c | 4 +++- libiberty/testsuite/demangle-expected | 6 ++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/libiberty/cp-demangle.c b/libiberty/cp-demangle.c index 12093cc..44a0a9b 100644 --- a/libiberty/cp-demangle.c +++ b/libiberty/cp-deman

  1   2   >