[Bug ipa/66705] New: [5/6 Regression] section is missing linker error with -flto -fipa-pta

[Bug ipa/66705] New: [5/6 Regression] section is missing linker error with -flto -fipa-pta

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

            Bug ID: 66705
           Summary: [5/6 Regression] section is missing linker error with
                    -flto -fipa-pta
           Product: gcc
           Version: 6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: ipa
          Assignee: unassigned at gcc dot gnu.org
          Reporter: trippels at gcc dot gnu.org
                CC: hubicka at gcc dot gnu.org
  Target Milestone: ---

For example:

markus@x4 ~ % g++ -O2 -flto -fipa-pta -w tramp3d-v4.cpp
lto1: fatal error: /tmp/ccIP6ytn.ltrans0.o: section _ZTVN6Smarts8RunnableE is
missing
compilation terminated.
lto-wrapper: fatal error: /usr/x86_64-pc-linux-gnu/gcc-bin/5.1.1/g++ returned 1
exit status
compilation terminated.
/usr/lib/gcc/x86_64-pc-linux-gnu/5.1.1/../../../../x86_64-pc-linux-gnu/bin/ld:
fatal error: lto-wrapper failed
collect2: error: ld returned 1 exit status

markus@x4 ~ % c++filt _ZTVN6Smarts8RunnableE
vtable for Smarts::Runnable

Reducing...

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

--- Comment #1 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
trippels@gcc75 ~ % cat tramp3d-v4.ii
class A {
public:
  A();
};
int a = 0;
void foo() {
  a = 0;
  A b;
  for (; a;)
    ;
}

trippels@gcc75 ~ % g++ -r -nostdlib -O2 -flto -flto-partition=max -fipa-pta
tramp3d-v4.ii
lto1: fatal error: /home/trippels/tmp/cccC3PXI.ltrans0.o: section a is missing
compilation terminated.

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2015-06-30
   Target Milestone|---                         |5.2
     Ever confirmed|0                           |1

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Confirmed.

Breakpoint 1, fatal_error (loc=0, gmsgid=0x1783549 "%s: section %s is missing")
    at /space/rguenther/src/svn/gcc-5-branch/gcc/diagnostic.c:1203
1203      va_start (ap, gmsgid);
Missing separate debuginfos, use: zypper install
libgmp10-debuginfo-6.0.0-71.1.x86_64 libisl13-debuginfo-0.14-25.2.x86_64
libmpc3-debuginfo-1.0.2-38.2.x86_64 libmpfr4-debuginfo-3.1.2-3.1.2.x86_64
(gdb) up
#1  0x0000000000fe0387 in varpool_node::get_constructor (this=0x7ffff6ac4000)
    at /space/rguenther/src/svn/gcc-5-branch/gcc/varpool.c:328
328                      name);
(gdb) l
323       data = lto_get_section_data (file_data, LTO_section_function_body,
324                                    name, &len);
325       if (!data)
326         fatal_error (input_location, "%s: section %s is missing",
327                      file_data->file_name,
328                      name);

(gdb) up
#2  0x0000000000eacc07 in create_variable_info_for (decl=0x7ffff68d7900, 
    name=0x1763482 "NULL")
    at /space/rguenther/src/svn/gcc-5-branch/gcc/tree-ssa-structalias.c:5808
5808              if (vnode->get_constructor ()
(gdb) l
5803              if (!vnode->all_refs_explicit_p ())
5804                make_copy_constraint (vi, nonlocal_id);
5805
5806              /* If this is a global variable with an initializer and we
are in
5807                 IPA mode generate constraints for it.  */
5808              if (vnode->get_constructor ()
5809                  && vnode->definition)

so IPA PTA needs to guard ->get_constructor () somehow?  That is, it
doesn't return NULL_TREE when it cant' get its hands on it?

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
(gdb) p debug_tree (decl)
 <var_decl 0x7ffff68d7900 a
    type <integer_type 0x7ffff68d0690 int public SI
        size <integer_cst 0x7ffff68ccdf8 constant 32>
        unit size <integer_cst 0x7ffff68cce10 constant 4>
        align 32 symtab 0 alias set -1 canonical type 0x7ffff68d0690 precision
32 min <integer_cst 0x7ffff68ccdb0 -2147483648> max <integer_cst 0x7ffff68ccdc8
2147483647>
        pointer_to_this <pointer_type 0x7ffff68e77e0>>
    public external SI file t.ii line 5 col 5 size <integer_cst 0x7ffff68ccdf8
32> unit size <integer_cst 0x7ffff68cce10 4>
    align 32 context <translation_unit_decl 0x7ffff7ff11e0 D.3949> initial
<error_mark 0x7ffff68ccba0>>

so it's just a global (in another partition I suppose) ...

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|5.2                         |5.3

--- Comment #4 from Richard Biener <rguenth at gcc dot gnu.org> ---
GCC 5.2 is being released, adjusting target milestone to 5.3.

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |gjl at gcc dot gnu.org

--- Comment #5 from Richard Biener <rguenth at gcc dot gnu.org> ---
*** Bug 67428 has been marked as a duplicate of this bug. ***

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot gnu.org

--- Comment #6 from Richard Biener <rguenth at gcc dot gnu.org> ---
I have a fix.

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

--- Comment #7 from Richard Biener <rguenth at gcc dot gnu.org> ---
Author: rguenth
Date: Thu Sep  3 09:24:51 2015
New Revision: 227430

URL: https://gcc.gnu.org/viewcvs?rev=227430&root=gcc&view=rev
Log:
2015-09-03  Richard Biener  <rguenther@suse.de>

        PR ipa/66705
        * tree-ssa-structalias.c (ctor_for_analysis): New function.
        (create_variable_info_for_1): Use ctor_for_analysis instead
        of get_constructor.
        (create_variable_info_for): Likewise.

        * g++.dg/lto/pr66705_0.C: New testcase.

Added:
    trunk/gcc/testsuite/g++.dg/lto/pr66705_0.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-ssa-structalias.c

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

--- Comment #8 from Richard Biener <rguenth at gcc dot gnu.org> ---
Author: rguenth
Date: Thu Sep  3 09:25:35 2015
New Revision: 227431

URL: https://gcc.gnu.org/viewcvs?rev=227431&root=gcc&view=rev
Log:
2015-09-03  Richard Biener  <rguenther@suse.de>

        PR ipa/66705
        * tree-ssa-structalias.c (ctor_for_analysis): New function.
        (create_variable_info_for_1): Use ctor_for_analysis instead
        of get_constructor.
        (create_variable_info_for): Likewise.

        * g++.dg/lto/pr66705_0.C: New testcase.

Added:
    branches/gcc-5-branch/gcc/testsuite/g++.dg/lto/pr66705_0.C
Modified:
    branches/gcc-5-branch/gcc/ChangeLog
    branches/gcc-5-branch/gcc/testsuite/ChangeLog
    branches/gcc-5-branch/gcc/tree-ssa-structalias.c

[Bug ipa/66705] [5/6 Regression] section is missing linker error with -flto -fipa-pta

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66705

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
      Known to fail|                            |5.2.0

--- Comment #9 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed.