https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #30 from Jan Hubicka hubicka at gcc dot gnu.org ---
Author: hubicka
Date: Mon Feb 9 21:06:35 2015
New Revision: 220549
URL: https://gcc.gnu.org/viewcvs?rev=220549root=gccview=rev
Log:
PR ipa/61548
* tree-emutls.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Jan Hubicka hubicka at gcc dot gnu.org changed:
What|Removed |Added
Status|REOPENED|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #29 from Hans-Peter Nilsson hp at gcc dot gnu.org ---
(In reply to Hans-Peter Nilsson from comment #28)
(In reply to Jan Hubicka from comment #27)
Does the following patch fix the problem?
Yes! Full regtest is underway but this
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #25 from Hans-Peter Nilsson hp at gcc dot gnu.org ---
Created attachment 34695
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34695action=edit
Assembly file showing the duplicate label
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Hans-Peter Nilsson hp at gcc dot gnu.org changed:
What|Removed |Added
Status|RESOLVED|REOPENED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #26 from Jan Hubicka hubicka at gcc dot gnu.org ---
Hmm, emutls_foo is added to symtab twice and therefore it is also output twice.
__emutls_v.foo/6 (__emutls_v.foo) @0x76973300
Type: variable
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #27 from Jan Hubicka hubicka at gcc dot gnu.org ---
Does the following patch fix the problem?
Index: tree-emutls.c
===
--- tree-emutls.c (revision 220509)
+++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #28 from Hans-Peter Nilsson hp at gcc dot gnu.org ---
(In reply to Jan Hubicka from comment #27)
Does the following patch fix the problem?
Yes! Full regtest is underway but this particular FAIL is fixed. Thanks.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #23 from ktkachov at gcc dot gnu.org ---
(In reply to Jan Hubicka from comment #22)
Trevor, seems I missed your patch. Sorry for that :(
I committed the last part then and lets close the PR
Thanks!
The testcase doesn't ICE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #18 from Jan Hubicka hubicka at gcc dot gnu.org ---
Author: hubicka
Date: Thu Feb 5 03:57:32 2015
New Revision: 220432
URL: https://gcc.gnu.org/viewcvs?rev=220432root=gccview=rev
Log:
PR ipa/61548
* varpool.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #19 from Jan Hubicka hubicka at gcc dot gnu.org ---
Author: hubicka
Date: Thu Feb 5 04:01:25 2015
New Revision: 220433
URL: https://gcc.gnu.org/viewcvs?rev=220433root=gccview=rev
Log:
PR ipa/61548
* ipa.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #17 from Jan Hubicka hubicka at gcc dot gnu.org ---
Created attachment 34673
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34673action=edit
Proposed fix
Ok, there seems to be 3 issues ;) First is that emutls does not resolve
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Jan Hubicka hubicka at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #21 from Jan Hubicka hubicka at gcc dot gnu.org ---
Author: hubicka
Date: Thu Feb 5 04:45:41 2015
New Revision: 220434
URL: https://gcc.gnu.org/viewcvs?rev=220434root=gccview=rev
Log:
PR ipa/61548
* tree-emultls.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #20 from tbsaunde at tbsaunde dot org ---
Created attachment 34673
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34673action=edit
Proposed fix
Ok, there seems to be 3 issues ;) First is that emutls does not resolve alias
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #16 from tbsaunde at gcc dot gnu.org ---
(In reply to Jan Hubicka from comment #15)
I think it is best to modify the remove_unreachable_nodes loop to first
remove aliases before removing their target...
how is this related? the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #15 from Jan Hubicka hubicka at gcc dot gnu.org ---
I think it is best to modify the remove_unreachable_nodes loop to first remove
aliases before removing their target...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #14 from tbsaunde+mozilla at tbsaunde dot org ---
On Mon, Jan 26, 2015 at 03:56:05AM +, hubicka at ucw dot cz wrote:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #13 from Jan Hubicka hubicka at ucw dot cz ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #13 from Jan Hubicka hubicka at ucw dot cz ---
So, what is here is:
emutls calls varpool_node::create_alias(decl(emutls_v.bar),
decl(emutls_v.foo)) which sets alias = true and alias_target =
node(emutls_v.foo) on the node for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
tbsaunde at gcc dot gnu.org changed:
What|Removed |Added
CC||tbsaunde at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Status|WAITING |ASSIGNED
--- Comment #11
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Jan Hubicka hubicka at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|WAITING
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Ramana Radhakrishnan ramana at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|WAITING
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Status|WAITING |NEW
--- Comment #9 from
26 matches
Mail list logo
