Re: Fix PR 65177: diamonds are not valid execution threads for jump threading

2015-03-22 Thread Jeff Law
On 03/19/2015 01:54 PM, Sebastian Pop wrote: Richard Biener wrote: please instead fixup after copy_bbs in duplicate_seme_region. Thanks for the review. Attached patch that does not modify copy_bbs. Fixes make check in hmmer and make check RUNTESTFLAGS=tree-ssa.exp Full bootstrap and regtest

Re: Fix PR 65177: diamonds are not valid execution threads for jump threading

2015-03-22 Thread Jeff Law
On 03/19/2015 03:16 AM, Richard Biener wrote: On Wed, Mar 18, 2015 at 11:35 PM, Sebastian Pop wrote: Hi, the attached patch fixes PR 65177 in which the code generator of FSM jump thread would create a diamond on the copied path: see https://gcc.gnu.org/PR65177#c18 for a detailed description.

Re: [PATCH][RFA] [PR rtl-optimization/64317] Enhance postreload-gcse.c to eliminate more redundant loads

2015-03-22 Thread Jeff Law
On 03/18/2015 12:19 AM, Andrew Pinski wrote: On Tue, Mar 17, 2015 at 11:27 AM, Jeff Law wrote: On 03/17/2015 04:35 AM, Richard Biener wrote: I'll test both. In the common case, the cost is going to be the basic bookkeeping so that we can compute the transparent property. The actual computat

Re: [PATCH][RFA] [PR rtl-optimization/64317] Enhance postreload-gcse.c to eliminate more redundant loads

2015-03-22 Thread Jeff Law
On 03/16/2015 01:27 PM, Jakub Jelinek wrote: On Wed, Mar 11, 2015 at 03:30:36PM -0600, Jeff Law wrote: +#ifndef GCC_GCSE__COMMONH +#define GCC_GCSE__COMMONH GCC_GCSE_COMMON_H instead? @@ -1308,8 +1396,19 @@ gcse_after_reload_main (rtx f ATTRIBUTE_UNUSED) if (expr_table->elements () > 0)

Re: [PATCH][RFA] [PR rtl-optimization/64317] Enhance postreload-gcse.c to eliminate more redundant loads

2015-03-22 Thread Jeff Law
On 03/16/2015 01:27 PM, Jakub Jelinek wrote: On Wed, Mar 11, 2015 at 03:30:36PM -0600, Jeff Law wrote: +#ifndef GCC_GCSE__COMMONH +#define GCC_GCSE__COMMONH GCC_GCSE_COMMON_H instead? @@ -1308,8 +1396,19 @@ gcse_after_reload_main (rtx f ATTRIBUTE_UNUSED) if (expr_table->elements () > 0)

Re: [wwwdocs] Describe the new way command line options are handled with LTO

2015-03-22 Thread Jan Hubicka
> How about this for a copy-edited version of the new text? Sounds good to me. Thanks, Sandra. Honza > >Command-line optimization and target options are now streamed on > a per-function basis and honored by the link-time optimizer. > This change makes the link-time optimi

Re: [wwwdocs] Describe the new way command line options are handled with LTO

2015-03-22 Thread Sandra Loosemore
On 03/22/2015 08:26 PM, Jan Hubicka wrote: > Hi, > this is my honest attempt to explain how command line options works with LTO > in current compiler and how to deal with these sanely. > > It is bit of mess, but an improvement over past releases. We finished the > transition to per-function attri

New French PO file for 'cpplib' (version 5.1-b20150208)

2015-03-22 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the French team of translators. The file is available at: http://translationproject.org/latest/cpplib/fr.po (This file, 'cpplib-5.1-b20150208.

[wwwdocs] Describe the new way command line options are handled with LTO

2015-03-22 Thread Jan Hubicka
Hi, this is my honest attempt to explain how command line options works with LTO in current compiler and how to deal with these sanely. It is bit of mess, but an improvement over past releases. We finished the transition to per-function attributes. In GCC 6 I plan to add per-variable counterpart

[patch, doc] hyphenate "command-line" when used as an adjective

2015-03-22 Thread Sandra Loosemore
I've checked in this patch to take care of another documentation glitch from my backlog I spotted while working on something else. "Command line", unhyphenated, is a noun, but when used as an adjective phrase immediately before the noun it modifies, e.g. "command-line option", it should be hyp

Fix ICEs on ODR violating programs and improve ODR mismatch diagnostic

2015-03-22 Thread Jan Hubicka
Hi, this patch fixes ODR violation warnings on Chromium and firefox. I went through all of them and found two false positives. I also fixes two ICEs that are triggered by testcases martin found while delta reducing. I tested the patch on firefox,chromium and libreoffice. Martin gave it a try on

Re: [Patch, Fortran, 4.8/4.9/5 Regression] PR59513 READ or WRITE not allowed after EOF

2015-03-22 Thread Jerry DeLisle
On 03/22/2015 08:47 AM, Janne Blomqvist wrote: On Sat, Mar 21, 2015 at 12:24 AM, Jerry DeLisle wrote: The attached patch allows the attempt to READ or WRITE after an EOF for legacy code. The runtime error is suppressed for -std=legacy and -std=gnu. For standard conformance the error is retaine

Re: [PATCH] Speed-up IPA ICF by enhanced hash values

2015-03-22 Thread Martin Liška
On 03/19/2015 09:42 PM, Jan Hubicka wrote: > >> diff --git a/gcc/ipa-icf.c b/gcc/ipa-icf.c >> index f68d23c..b8e3aa4 100644 >> --- a/gcc/ipa-icf.c >> +++ b/gcc/ipa-icf.c >> @@ -557,6 +557,69 @@ sem_function::equals_wpa (sem_item *item, >>return true; >> } >> >> +/* Update hash by address se

Re: [Patch, Fortran, 4.8/4.9/5 Regression] PR59513 READ or WRITE not allowed after EOF

2015-03-22 Thread Jerry DeLisle
On 03/22/2015 08:47 AM, Janne Blomqvist wrote: On Sat, Mar 21, 2015 at 12:24 AM, Jerry DeLisle wrote: The attached patch allows the attempt to READ or WRITE after an EOF for legacy code. The runtime error is suppressed for -std=legacy and -std=gnu. For standard conformance the error is retaine

Fix ipa-comdats WRT thunks

2015-03-22 Thread Jan Hubicka
Hi, this patch fixes ICE when buliding chromium with LTO and on ipa-pure-const patch. The patch exposes semi-latent bug in ipa-comdats that is trying to separate thunk from its target into different sections. This will lead to unrecognized insns at PPC and others that do not support tail call acr

Update Danny Smith's entry in contrib.tex

2015-03-22 Thread Gerald Pfeifer
This is something I meant to do back in 2010. Better late than never, I guess. Gerald 2015-03-22 Dave Korn Gerald Pfeifer * doc/contrib.texi (Contributors): Update entry for Danny Smith. Index: doc/contrib.texi ==

Re: [Patch, Fortran, 4.8/4.9/5 Regression] PR59513 READ or WRITE not allowed after EOF

2015-03-22 Thread Janne Blomqvist
On Sat, Mar 21, 2015 at 12:24 AM, Jerry DeLisle wrote: > The attached patch allows the attempt to READ or WRITE after an EOF for > legacy code. The runtime error is suppressed for -std=legacy and -std=gnu. > For standard conformance the error is retained as is now. Since it's a standard violatio

Re: [Patch, Fortran, 4.8/4.9/5 Regression] PR59513 READ or WRITE not allowed after EOF

2015-03-22 Thread Dominique Dhumieres
Dear Jerry, IMO the patch is in the obvious range, but needs to document the extension and may be a test case. Cheers, Dominique

Re: [Ping, Patch 2/2, v3, Fortran, pr60322 a.o.] [OOP] Incorrect bounds on polymorphic dummy array

2015-03-22 Thread Dominique Dhumieres
Dear Andre, If I am not mistaken, this patch make the following test (pr57305, second attachment): subroutine add_element_poly(a,e) use iso_c_binding class(*),allocatable,intent(inout),target :: a(:) class(*),intent(in),target :: e class(*),allocatable,target :: tmp(:

Re: [patch, nios2] implement TARGET_ASM_OUTPUT_MI_THUNK

2015-03-22 Thread Eric Botcazou
> The nios2 back end didn't previously implement TARGET_ASM_OUTPUT_MI_THUNK. Then backends.html needs to be adjusted, done thusly, installed. -- Eric BotcazouIndex: backends.html === RCS file: /cvs/gcc/wwwdocs/htdocs/backends.html,v

Re: [Patch, Fortran] Reject unsupported coarray communication

2015-03-22 Thread Dominique Dhumieres
Dear Tobias, The test gfortran.dg/coarray/coindexed_3.f90 compiles without error, see https://gcc.gnu.org/ml/gcc-testresults/2015-03/msg02446.html. TIA Dominique