https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Jan Hubicka changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #30 from Jan Hubicka ---
Author: hubicka
Date: Mon Feb 9 21:06:35 2015
New Revision: 220549
URL: https://gcc.gnu.org/viewcvs?rev=220549&root=gcc&view=rev
Log:
PR ipa/61548
* tree-emutls.c (ipa_lower_emutls): Avoid duplicates
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #29 from Hans-Peter Nilsson ---
(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 particular FAIL i
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #28 from Hans-Peter Nilsson ---
(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 #27 from Jan Hubicka ---
Does the following patch fix the problem?
Index: tree-emutls.c
===
--- tree-emutls.c (revision 220509)
+++ tree-emutls.c (working cop
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #26 from Jan Hubicka ---
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 definition analyzed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #25 from Hans-Peter Nilsson ---
Created attachment 34695
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34695&action=edit
Assembly file showing the duplicate label
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Hans-Peter Nilsson changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
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 an
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #21 from Jan Hubicka ---
Author: hubicka
Date: Thu Feb 5 04:45:41 2015
New Revision: 220434
URL: https://gcc.gnu.org/viewcvs?rev=220434&root=gcc&view=rev
Log:
PR ipa/61548
* tree-emultls.c (new_emutls_decl): Resolve alias af
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=34673&action=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
--- Comment #19 from Jan Hubicka ---
Author: hubicka
Date: Thu Feb 5 04:01:25 2015
New Revision: 220433
URL: https://gcc.gnu.org/viewcvs?rev=220433&root=gcc&view=rev
Log:
PR ipa/61548
* ipa.c (symbol_table::remove_unreachable_nodes): F
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #18 from Jan Hubicka ---
Author: hubicka
Date: Thu Feb 5 03:57:32 2015
New Revision: 220432
URL: https://gcc.gnu.org/viewcvs?rev=220432&root=gcc&view=rev
Log:
PR ipa/61548
* varpool.c (varpool_node::remove): Fix order of va
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #17 from Jan Hubicka ---
Created attachment 34673
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34673&action=edit
Proposed fix
Ok, there seems to be 3 issues ;) First is that emutls does not resolve alias
after introducing it
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 ta
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #15 from Jan Hubicka ---
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 ---
> > So, what is he
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
--- Comment #13 from Jan Hubicka ---
> 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 "emutls_v.bar",
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 fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|WAITING
--- Comment #10 from Jan Hubicka
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Ramana Radhakrishnan changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
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 ktk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548
Richard Biener changed:
What|Removed |Added
Status|ASSIGNED|WAITING
Component|regression
26 matches
Mail list logo
