On Fri, Mar 7, 2014 at 12:29 AM, Paulo Matos <pa...@matos-sorge.com> wrote: > On 06/03/14 11:19, Richard Biener wrote: >> >> On Wed, Mar 5, 2014 at 12:55 PM, Paulo Matos <pa...@matos-sorge.com> >> wrote: >>> >>> On 05/03/2014 11:51, Richard Biener wrote: >>>> >>>> >>>> On Wed, Mar 5, 2014 at 12:43 PM, Dominique Dhumieres >>>> <domi...@lps.ens.fr> >>>> wrote: >>>>> >>>>> >>>>> >>>>> Revision 208312 causes >>>>> >>>>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60427 >>>> >>>> >>>> >>>> Uhm. pointer comparison against double_type_node ... >>>> >>>> I'd say we want to revert the patch. Paulo, please do that. Let's >>>> do the alternate approach of marking -fshort-double eligible for LTO >>>> as well and handle it there properly. >>>> >>> >>> Sure, I will prepare a new patch and post it for approval by the end of >>> the >>> day. >>> >>> Apologies for the regression. >> >> >> I have reverted the patch for now. >> >> Richard. >> > > > Please find new patch attached. I have enabled LTO for short-double and > passed flag_short_double to build_common_tree_nodes. > > I have tested this for C on powerpc-eabipse (target for which we are fixing > pr55113), and C/Fortran on a x86_64. Saw no regressions. > > OK to commit?
Ok. Thanks, Richard. > gcc/c-family/ > 2014-03-06 Paulo Matos <pa...@matos-sorge.com> > > * c.opt: Enable LTO FE for fshort-double. > > gcc/lto/ > 2014-03-06 Paulo Matos <pa...@matos-sorge.com> > > * lto-lang.c (lto_init): Pass flag_short_double to > build_common_tree_nodes. > > gcc/testsuite/ > 2014-03-06 Paulo Matos <pa...@matos-sorge.com> > > * gcc.dg/lto/pr55113_0.c: New testcase. > > > -- > Paulo Matos
