[Bug other/82352] [7/8 Regression] comdat-local function called by void h::i() outside its comdat

2018-01-04 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82352

--- Comment #4 from Martin Liška  ---
Author: marxin
Date: Thu Jan  4 08:54:17 2018
New Revision: 256226

URL: https://gcc.gnu.org/viewcvs?rev=256226=gcc=rev
Log:
Be careful about comdat boundary in ICF (PR ipa/82352).

2018-01-04  Martin Liska  

PR ipa/82352
* ipa-icf.c (sem_function::merge): Do not cross comdat boundary.
2018-01-04  Martin Liska  

PR ipa/82352
* g++.dg/ipa/pr82352.C: New test.

Added:
trunk/gcc/testsuite/g++.dg/ipa/pr82352.C
Modified:
trunk/gcc/ChangeLog
trunk/gcc/ipa-icf.c
trunk/gcc/testsuite/ChangeLog

[Bug other/82352] [7/8 Regression] comdat-local function called by void h::i() outside its comdat

2018-01-03 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82352

--- Comment #3 from Martin Liška  ---
> Can you please the patch for original test-case? Not the reduced one?

Can you please test the patch for original test-case? Not the reduced one?

[Bug other/82352] [7/8 Regression] comdat-local function called by void h::i() outside its comdat

2018-01-03 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82352

--- Comment #2 from Martin Liška  ---
Created attachment 43012
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43012=edit
Untested patch

Can you please the patch for original test-case? Not the reduced one?

[Bug other/82352] [7/8 Regression] comdat-local function called by void h::i() outside its comdat

2017-12-14 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82352

Richard Biener  changed:

   What|Removed |Added

   Priority|P3  |P2

[Bug other/82352] [7/8 Regression] comdat-local function called by void h::i() outside its comdat

2017-09-29 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82352

Martin Liška  changed:

   What|Removed |Added

   Keywords||ice-on-valid-code
 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2017-09-29
   Assignee|unassigned at gcc dot gnu.org  |marxin at gcc dot 
gnu.org
   Target Milestone|--- |7.3
Summary|link error 'defined in  |[7/8 Regression]
   |discarded section'  |comdat-local function
   ||called by void h::i()
   ||outside its comdat
 Ever confirmed|0   |1