[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-09 Thread derodat at adacore dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #18 from Pierre-Marie de Rodat --- Excellent, thank you for the feedback!

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-09 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #17 from Martin Jambor --- Yes, 521.wrf_r indeed now builds fine with LTO. Thanks everyone for a quick fix.

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-09 Thread derodat at adacore dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 Pierre-Marie de Rodat changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-09 Thread pmderodat at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #15 from pmderodat at gcc dot gnu.org --- Author: pmderodat Date: Fri Feb 9 14:02:37 2018 New Revision: 257526 URL: https://gcc.gnu.org/viewcvs?rev=257526&root=gcc&view=rev Log: DWARF: no location for non-definition DECLs with non-tr

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-09 Thread derodat at adacore dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #14 from Pierre-Marie de Rodat --- (In reply to Pierre-Marie de Rodat from comment #13) > IIRC, DECL_VALUE_EXPR is something like MEM_REF (-1). Yes, I’ll give it a > try… I submitted a candidate patch: https://gcc.gnu.org/ml/gcc-patc

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread derodat at adacore dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #13 from Pierre-Marie de Rodat --- (In reply to rguent...@suse.de from comment #11) > This means you should be able to arrange for this > to be emitted by tree_add_const_value_attribute_for_decl as well, no? I don’t know… I have to d

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #12 from Jakub Jelinek --- I believe DWARF doesn't have a way to push address of a DIE (if it has one), only can use DW_OP_addr which pushes an address of a relocatable symbol rather than a DIE.

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #11 from rguenther at suse dot de --- On Thu, 8 Feb 2018, derodat at adacore dot com wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 > > --- Comment #10 from Pierre-Marie de Rodat --- > (In reply to Richard Biener from

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread derodat at adacore dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #10 from Pierre-Marie de Rodat --- (In reply to Richard Biener from comment #8) > Yeah, setting DECL_PRESERVE_P would make it possible to add the > location late since we'd not remove the decl. Currently it's simply > gone and the ea

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 Richard Biener changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #9

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #8 from rguenther at suse dot de --- On Thu, 8 Feb 2018, derodat at adacore dot com wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 > > Pierre-Marie de Rodat changed: > >What|Removed

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread derodat at adacore dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 Pierre-Marie de Rodat changed: What|Removed |Added CC||derodat at adacore dot com --- C

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #6 from Richard Biener --- In the Fortran case the DECL_VALUE_EXPR is COMPONENT_REF and rtl_for_decl_location gets to a location via loc_list_from_tree which eventually looks at DECL_VALUE_EXPR. I'm quite sure the rev. is responsible

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-08 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 Richard Biener changed: What|Removed |Added CC||pmderodat at gcc dot gnu.org --- Commen

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-07 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #4 from Richard Biener --- MODULE MODULE_RA_GFDLETA REAL , SAVE :: EM1(28,180) REAL ,SAVE, DIMENSION(5040):: EM1V EQUIVALENCE (EM1V(1),EM1(1,1)) END MODULE module_RA_GFDLETA

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-07 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 --- Comment #3 from Richard Biener --- So the issue here is that the early debug generated for module_ra_gfdleta.F90 contains relocations to text symbols. That's a no-no. So this is reproducible with just compiling this file with -flto -g, the

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-07 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 Richard Biener changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug lto/84213] [8 Regression] 521.wrf_r from SPEC 2017 fails to build (link) with LTO

2018-02-06 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84213 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|