[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW Known to work|

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 Richard Biener changed: What|Removed |Added CC||miles at gnu dot org --- Comment #9 from

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 Jan Hubicka changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at g

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 Richard Biener changed: What|Removed |Added Priority|P3 |P1 --- Comment #11 from Richard Biener

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #12

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 --- Comment #13 from Jakub Jelinek --- --- gcc/symtab.c.jj2014-03-21 22:23:43.0 +0100 +++ gcc/symtab.c2014-04-09 17:37:40.709523997 +0200 @@ -1312,9 +1312,13 @@ symtab_get_symbol_partitioning_class (sy /* Linker discardable symb

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 --- Comment #14 from Jan Hubicka --- OK, the problem is that one comdat group has two functions: _ZNK19MutableIntegerValue18isValidNativeValueEi/0 (isValidNativeValue) @0x76adfe18 Type: function definition analyzed Visibility: forced_by_ab

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 Jan Hubicka changed: What|Removed |Added CC||jason at redhat dot com --- Comment #15 fro

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 --- Comment #16 from Jason Merrill --- (In reply to Jan Hubicka from comment #14) > It doesn't seem to make sense to keep function > but optimize out the thunk as we would do now (even w/o LTO) Right. When we emit a function, we need to also emi

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 --- Comment #17 from Jakub Jelinek --- (In reply to Jan Hubicka from comment #15) > This patch fixes the ICE by copying forced_by_abi as part of cgraph fixup in > ipa visibility. I would like Jason to comment on this. I think fix at C++ > FE side

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 --- Comment #18 from Jakub Jelinek --- By the C++ FE change, do you mean something like: --- gcc/cp/method.c.jj2014-03-27 08:06:11.0 +0100 +++ gcc/cp/method.c2014-04-10 10:59:36.226288999 +0200 @@ -387,6 +387,7 @@ use_thunk (tree t

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 --- Comment #19 from Jakub Jelinek --- Created attachment 32581 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32581&action=edit gcc49-pr60567.patch The ipa.c version (changing node->forced_by_abi instead of decl_node->forced_by_abi), also b

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 --- Comment #20 from Jakub Jelinek --- (In reply to Jakub Jelinek from comment #19) > Created attachment 32581 [details] > gcc49-pr60567.patch > > The ipa.c version (changing node->forced_by_abi instead of > decl_node->forced_by_abi), also bootst

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 --- Comment #21 from Jakub Jelinek --- Author: jakub Date: Thu Apr 10 18:57:48 2014 New Revision: 209280 URL: http://gcc.gnu.org/viewcvs?rev=209280&root=gcc&view=rev Log: PR lto/60567 * ipa.c (function_and_variable_visibility): Copy force

[Bug lto/60567] [4.9 Regression] lto1 ICE in add_symbol_to_partition, at lto/lto-partition.c:233 with -fno-use-linker-plugin

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60567 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---