Re: [PATCH] Reenable CSE of non-volatile inline asm (PR rtl-optimization/63637)

2015-01-15 Thread Richard Biener
On Thu, 15 Jan 2015, Jakub Jelinek wrote: On Thu, Jan 15, 2015 at 07:46:18AM +0100, Richard Biener wrote: That last line means the compiler is free to delete a non-volatile asm with a memory clobber if that asm is not needed for dataflow. Or that is how I read it; it is trying to

[PATCH] Avoid -Werror=format-security errors in libcpp

2015-01-15 Thread Jakub Jelinek
Hi! With the addition of build libcpp, my build failed because of distro default flags of -Werror=format-security. The first hunk is I think no big deal, making it const makes the warning go away. The second hunk is more controversial, as even making message const doesn't help with the warning.

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-15 Thread Jakub Jelinek
On Thu, Jan 15, 2015 at 12:17:44PM +0100, Richard Biener wrote: On Thu, Jan 15, 2015 at 12:10 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! I ran into -Werror=maybe-uninitialized errors during profiledbootstrap (../configure --enable-languages=c,c++ --enable-checking=release; make -j16

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-15 Thread Richard Biener
On Thu, Jan 15, 2015 at 12:29 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Jan 15, 2015 at 12:17:44PM +0100, Richard Biener wrote: On Thu, Jan 15, 2015 at 12:10 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! I ran into -Werror=maybe-uninitialized errors during profiledbootstrap

Re: [Patch, Fortran, F03] PR 58023: ICE on invalid with bad PPC declaration

2015-01-15 Thread Janus Weil
Your patch at https://gcc.gnu.org/ml/fortran/2015-01/txtThsA1zTNFd.txt looks very similar to the Mikael's one for pr58023 at https://gcc.gnu.org/bugzilla/attachment.cgi?id=30633 with retval replaved with success. Was it intended? Nope. I wasn't even aware of that patch. To which PR is it

[PATCH] Fix darwin FAIL of gcc.dg/lto/pr64415_0.c

2015-01-15 Thread Richard Biener
Committed. Richard. 2015-01-15 Richard Biener rguent...@suse.de PR lto/64415 * gcc.dg/lto/pr64415_0.c: Skip on darwin. Index: gcc/testsuite/gcc.dg/lto/pr64415_0.c === --- gcc/testsuite/gcc.dg/lto/pr64415_0.c

Re: [Patch, Fortran, F03] PR 58023: ICE on invalid with bad PPC declaration

2015-01-15 Thread Dominique Dhumieres
Hi Janus, Your patch at https://gcc.gnu.org/ml/fortran/2015-01/txtThsA1zTNFd.txt looks very similar to the Mikael's one for pr58023 at https://gcc.gnu.org/bugzilla/attachment.cgi?id=30633 with retval replaved with success. Was it intended? Cheers, Dominique

[PATCH, CHKP, PR64363] Don't instrument functions we cannot copy

2015-01-15 Thread Ilya Enkovich
Hi, This patch is to fix https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64363. Patch disables instrumentation for functions we cannot clone correctly due to labels. Bootstrapped and checked on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Ilya -- gcc/ 2015-01-14 Ilya Enkovich

Re: [PATCH PR64434]

2015-01-15 Thread Richard Biener
On Thu, Jan 15, 2015 at 10:39 AM, Yuri Rumyantsev ysrum...@gmail.com wrote: Hi All, I did a change proposed by Richard - unconditionally allocate from the heap. Bootstrap and regression testing did not show any new failures. Is it OK for trunk? + if (!is_gimple_assign (stmt) +

[PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-15 Thread Jakub Jelinek
Hi! I ran into -Werror=maybe-uninitialized errors during profiledbootstrap (../configure --enable-languages=c,c++ --enable-checking=release; make -j16 profiledboostrap) before I hit a miscompilation I'm going to file. Is this ok for trunk, or do we want to work around them differently?

Re: [PATCH] Workaround -Wmaybe-uninitialized false positives during profiledbootstrap

2015-01-15 Thread Richard Biener
On Thu, Jan 15, 2015 at 12:10 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! I ran into -Werror=maybe-uninitialized errors during profiledbootstrap (../configure --enable-languages=c,c++ --enable-checking=release; make -j16 profiledboostrap) before I hit a miscompilation I'm going to file.

RE: [PATCH] Allow MIPS call-saved-{4-6}.c tests to correctly run for micromips

2015-01-15 Thread Matthew Fortune
I have tested this for both mips and micromips, and the tests now pass successfully. The ChangeLog and patch are below. Ok to commit? Since you had not got to committing this yet. I have added the micromips variants of the tests and committed your patch for you. Thanks for finding the

[PATCH,committed] Ensure options incompatible with micromips imply -mno-micromips

2015-01-15 Thread Matthew Fortune
A bit of housekeeping in mips.exp. Several test options are incompatible with micromips so this patch enforces no-micromips as required. The number of failures in mips.exp for -mmicromips is now much lower and primarily related to branch distance differences vs MIPS. Thanks, Matthew

Re: [PATCH] Reenable CSE of non-volatile inline asm (PR rtl-optimization/63637)

2015-01-15 Thread Jakub Jelinek
On Thu, Jan 15, 2015 at 09:13:30AM +0100, Jakub Jelinek wrote: differently. Just trying to grep for a few: glibc: ./sysdeps/alpha/bits/atomic.h:# define atomic_full_barrier() __asm (mb : : : memory); ./sysdeps/alpha/bits/atomic.h:# define atomic_read_barrier() __asm (mb : : : memory);

Re: [Patch] Missing plugin header files

2015-01-15 Thread Richard Biener
On Wed, Jan 14, 2015 at 10:43 PM, Steve Ellcey sell...@imgtec.com wrote: I tried compiling an empty plugin that just included gcc-plugin.h and plugin-version.h and found that these header files were included from gcc-plugin.h but not in the list of header files to be copied to the plugin

Re: [PATCH PR64434]

2015-01-15 Thread Yuri Rumyantsev
Hi All, I did a change proposed by Richard - unconditionally allocate from the heap. Bootstrap and regression testing did not show any new failures. Is it OK for trunk? ChangeLog 2015-01-15 Yuri Rumyantsev ysrum...@gmail.com PR tree-optimization/64434 * cfgexpand.c (reorder_operands): New

[AArch64] Add a new scheduling description for the ARM Cortex-A57 processor

2015-01-15 Thread James Greenhalgh
Hi, This patch implements a new scheduler model for the ARM Cortex-A57 processor. This model results in better code generation for the Cortex-A57 with a more mixed blend of instruction types, particularly when scheduling for the new instructions introduced in ARMv8-A. I haven't yet wired it up

[Ping] [C++ Patch] PR 58614

2015-01-15 Thread Paolo Carlini
Hi, pinging this patchlet... https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01926.html On 12/29/2014 03:54 PM, Paolo Carlini wrote: Hi, in this ICE on invalid, we crash during error recovery when maybe_adjust_types_for_deduction gets an elt which has TREE_TYPE (elt) == error_mark_node.

RE: [PATCH, RFC] LRA subreg handling

2015-01-15 Thread Robert Suchanek
Robert, can you look at reload.c::reload_inner_reg_of_subreg and verify that the comment just before its return statement is effectively the situation you're in. There are certainly cases where a SUBREG needs to be treated as an in-out operand. We walked through them eons ago when we were

[PATCH] Fix PR61743

2015-01-15 Thread Richard Biener
I am testing the following hack^Wpatch which makes us preserve range information in some very special case in PRE. This is important to get number of iteration analysis results in a way to enable unrolling of loops in some EEMBC testcase (I don't have access to that benchmark unless it is part

Re: [testsuite] PATCH: Add -fno-pie to gcc.target/i386/pr54445-2.c

2015-01-15 Thread Uros Bizjak
On Mon, Jan 12, 2015 at 1:34 AM, H.J. Lu hjl.to...@gmail.com wrote: Since PIE is also PIC, we need to add -fno-pie to avoid PIC. OK for trunk? Thanks. H.J. --- gcc/testsuite/gcc.target/i386/pr54445-2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 2015-01-11 H.J. Lu

Re: [testsuite] PATCH: Add -fno-pie to gcc.target/i386/pr54445-2.c

2015-01-15 Thread H.J. Lu
On Thu, Jan 15, 2015 at 02:02:51PM +0100, Uros Bizjak wrote: On Thu, Jan 15, 2015 at 2:00 PM, H.J. Lu hjl.to...@gmail.com wrote: gcc/testsuite/gcc.target/i386/pr54445-2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 2015-01-11 H.J. Lu hongjiu...@intel.com *

Re: [Patch, Fortran, F03] PR 58023: ICE on invalid with bad PPC declaration

2015-01-15 Thread Mikael Morin
Le 14/01/2015 19:30, Janus Weil a écrit : Hi Mikael, the attached patch fixes an ICE-on-invalid problem with procedure-pointer components by making sure that we continue resolving all components of a derived type, even after an error is thrown. Does the fonction return false as before,

PATCH: PR libitm/64360: libitm.c/stackundo.c fails with -fpic

2015-01-15 Thread H.J. Lu
Hi, libitm.c/stackundo.c fails with -fpic since test1 and test2 may be preempted with -fpic. This patch makes those 2 functions static. Tested on Linux/x86. OK for trunk? Thanks. H.J. diff --git a/libitm/ChangeLog b/libitm/ChangeLog index 74e2940..e46819c 100644 --- a/libitm/ChangeLog

Re: [Patch, Fortran, F03] PR 58023: ICE on invalid with bad PPC declaration

2015-01-15 Thread Dominique d'Humières
Le 15 janv. 2015 à 12:37, Janus Weil ja...@gcc.gnu.org a écrit : Your patch at https://gcc.gnu.org/ml/fortran/2015-01/txtThsA1zTNFd.txt looks very similar to the Mikael's one for pr58023 at https://gcc.gnu.org/bugzilla/attachment.cgi?id=30633 with retval replaved with success. Was it

Re: [testsuite] PATCH: Add -fno-pie to gcc.target/i386/pr54445-2.c

2015-01-15 Thread H.J. Lu
On Thu, Jan 15, 2015 at 4:44 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Jan 12, 2015 at 1:34 AM, H.J. Lu hjl.to...@gmail.com wrote: Since PIE is also PIC, we need to add -fno-pie to avoid PIC. OK for trunk? Thanks. H.J. --- gcc/testsuite/gcc.target/i386/pr54445-2.c | 2 +- 1 file

Re: [testsuite] PATCH: Add -fno-pie to gcc.target/i386/pr54445-2.c

2015-01-15 Thread Uros Bizjak
On Thu, Jan 15, 2015 at 2:00 PM, H.J. Lu hjl.to...@gmail.com wrote: gcc/testsuite/gcc.target/i386/pr54445-2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 2015-01-11 H.J. Lu hongjiu...@intel.com * gcc.target/i386/pr54445-2.c: Add -fno-pie. diff --git

Re: [PATCH, CHKP, PR64363] Don't instrument functions we cannot copy

2015-01-15 Thread Richard Biener
On Thu, Jan 15, 2015 at 12:46 PM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch is to fix https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64363. Patch disables instrumentation for functions we cannot clone correctly due to labels. Bootstrapped and checked on

Re: [testsuite] PATCH: Add -fno-pie to gcc.target/i386/pr54445-2.c

2015-01-15 Thread Uros Bizjak
On Thu, Jan 15, 2015 at 2:15 PM, H.J. Lu hjl.to...@gmail.com wrote: gcc/testsuite/gcc.target/i386/pr54445-2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 2015-01-11 H.J. Lu hongjiu...@intel.com * gcc.target/i386/pr54445-2.c: Add -fno-pie. diff --git

[PATCH, ARM, testsuite] Improve scd42-1.c for UAL

2015-01-15 Thread Tony Liu
Hi, This is the patch to improve the test case gcc.target/arm/scd42-1.c for both UAL and non-UAL. It now checks UAL format assembly code for Thumb1 and Thumb2 while non-UAL format assembly code for ARM mode. With this patch, the test passes for both cases. Thanks, Tony 2015-01-15 Tony Liu

Re: [PATCH] Fix darwin FAIL of gcc.dg/lto/pr64415_0.c

2015-01-15 Thread Richard Biener
On Thu, 15 Jan 2015, Richard Biener wrote: Committed. Turns out we can do better. Committed. Richard. 2015-01-15 Richard Biener rguent...@suse.de PR lto/64415 * gcc.dg/lto/pr64415_0.c: Re-enable for darwin with -Wl,-undefined,dynamic_lookup. Index:

Re: libgo patch committed: Update to Go 1.4

2015-01-15 Thread Richard Biener
On Thu, Jan 15, 2015 at 1:28 AM, Ian Lance Taylor i...@golang.org wrote: I've committed a patch to libgo to update it to the Go 1.4 release, except for the runtime package. Much of the runtime package was rewritten in Go, and it does not really affect users of the library, so I've postponed

Re: [Patch, Fortran, F03] PR 58023: ICE on invalid with bad PPC declaration

2015-01-15 Thread Mikael Morin
Le 15/01/2015 14:25, Mikael Morin a écrit : Le 15/01/2015 12:37, Janus Weil a écrit : Your patch at https://gcc.gnu.org/ml/fortran/2015-01/txtThsA1zTNFd.txt looks very similar to the Mikael's one for pr58023 at https://gcc.gnu.org/bugzilla/attachment.cgi?id=30633 with retval replaved with

Re: flatten expr.h (version 2)

2015-01-15 Thread Prathamesh Kulkarni
On 14 January 2015 at 12:14, Richard Biener rguent...@suse.de wrote: On Wed, 14 Jan 2015, Prathamesh Kulkarni wrote: On 14 January 2015 at 17:31, Richard Biener rguent...@suse.de wrote: On Wed, 14 Jan 2015, Prathamesh Kulkarni wrote: On 14 January 2015 at 11:16, Prathamesh Kulkarni

Re: [PATCH, CHKP, PR64363] Don't instrument functions we cannot copy

2015-01-15 Thread Ilya Enkovich
2015-01-15 16:07 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Thu, Jan 15, 2015 at 12:46 PM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch is to fix https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64363. Patch disables instrumentation for functions we cannot clone

Re: [PATCH][AARCH64]Fix syntax error in target selector -O2 for volatileloadpair-1.c volatileloadpair-2.c

2015-01-15 Thread Richard Earnshaw
On 15/01/15 14:10, Renlin Li wrote: Hi all, This is a simple fix to correct two typos (I believe) in the newly introduced volatileloadpair-1.c volatileloadpair-2.c test case. After the change, the test cases compiles and runs correctly. Okay to commit? Regards, Renlin Li

Re: [Patch, Fortran, F03] PR 58023: ICE on invalid with bad PPC declaration

2015-01-15 Thread Mikael Morin
Le 15/01/2015 12:37, Janus Weil a écrit : Your patch at https://gcc.gnu.org/ml/fortran/2015-01/txtThsA1zTNFd.txt looks very similar to the Mikael's one for pr58023 at https://gcc.gnu.org/bugzilla/attachment.cgi?id=30633 with retval replaved with success. Was it intended? Nope. I wasn't

Re: [PATCH, CHKP, PR64363] Don't instrument functions we cannot copy

2015-01-15 Thread Richard Biener
On Thu, Jan 15, 2015 at 2:51 PM, Ilya Enkovich enkovich@gmail.com wrote: 2015-01-15 16:07 GMT+03:00 Richard Biener richard.guent...@gmail.com: On Thu, Jan 15, 2015 at 12:46 PM, Ilya Enkovich enkovich@gmail.com wrote: Hi, This patch is to fix

[PATCH][AARCH64]Fix syntax error in target selector -O2 for volatileloadpair-1.c volatileloadpair-2.c

2015-01-15 Thread Renlin Li
Hi all, This is a simple fix to correct two typos (I believe) in the newly introduced volatileloadpair-1.c volatileloadpair-2.c test case. After the change, the test cases compiles and runs correctly. Okay to commit? Regards, Renlin Li gcc/testsuite/ChangeLog: 2015-01-15 Renlin Li

[PATCH][x86] Update s{r,l}li intrinsics.

2015-01-15 Thread Ilya Tocar
Hi, Looks like new ISA doc [1] renamed srli,slli intrinsics to bsrli,bslli. This patch adds b* versions, while keeping old srli for backward compatibility. OK for trunk? 1:https://software.intel.com/sites/default/files/managed/0d/53/319433-022.pdf ChangeLog: gcc/ *

Stage3 closing soon, call for patch pings

2015-01-15 Thread Jeff Law
Stage3 is closing rapidly. I've drained my queue of patches I was tracking for gcc-5.However, note that I don't track everything. If it's a patch for a backend, language other than C or seemingly has another maintainer that's engaged in review, then I haven't been tracking the patch.

Re: [PATCH][x86] Update s{r,l}li intrinsics.

2015-01-15 Thread Uros Bizjak
On Thu, Jan 15, 2015 at 3:17 PM, Ilya Tocar tocarip.in...@gmail.com wrote: Hi, Looks like new ISA doc [1] renamed srli,slli intrinsics to bsrli,bslli. This patch adds b* versions, while keeping old srli for backward compatibility. OK for trunk?

Re: [C++ Patch] PR 58614

2015-01-15 Thread Jason Merrill
OK. Jason

<    1   2