[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-16 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 Martin Liška changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-16 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #10 from Martin Liška --- Author: marxin Date: Fri Mar 16 14:53:45 2018 New Revision: 258596 URL: https://gcc.gnu.org/viewcvs?rev=258596&root=gcc&view=rev Log: Redirect reference in the symbol table (PR ipa/84833). 2018-03-16 Marti

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-14 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #9 from Martin Liška --- (In reply to Jakub Jelinek from comment #8) > Feel free to suggest it to binutils folks. Unfortunately, it seems I is > taken already. Done that: https://sourceware.org/bugzilla/show_bug.cgi?id=22967

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-14 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #8 from Jakub Jelinek --- Feel free to suggest it to binutils folks. Unfortunately, it seems I is taken already.

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-14 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #7 from Martin Liška --- > > nm is not really usable to determine if an IFUNC symbol is global or local, > i is printed regardless if it is global or local. Shouldn't nm distinguish there 2 variants to be more fine?

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-13 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #6 from Jakub Jelinek --- (In reply to Martin Liška from comment #5) > (In reply to Jakub Jelinek from comment #2) > > Just a note, as can be seen e.g. on > > static volatile int a; > > static int my_foo1 (void) { return 1; } > > stat

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-13 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #5 from Martin Liška --- (In reply to Jakub Jelinek from comment #2) > Just a note, as can be seen e.g. on > static volatile int a; > static int my_foo1 (void) { return 1; } > static int my_foo2 (void) { return 2; } > typedef int (*F)

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-13 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #4 from Martin Liška --- I can confirm the issue, there's missing edge to foo->baz and thus the symbol is removed. I'll fix that.

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-13 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #3 from Martin Liška --- > Another (though preexisting) bug is lack of optimization, if we have a call > or reference of a target_clones dispatcher in a function with the same > target_clones attribute (or maybe even just target attri

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-13 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 --- Comment #2 from Jakub Jelinek --- Just a note, as can be seen e.g. on static volatile int a; static int my_foo1 (void) { return 1; } static int my_foo2 (void) { return 2; } typedef int (*F) (void); static F resolve_foo (void) { if (a) return

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-13 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 Martin Liška changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at

[Bug ipa/84833] [8 Regression] target_clones regression since r251047

2018-03-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84833 Jakub Jelinek changed: What|Removed |Added Priority|P3 |P1 Status|UNCONFIRMED