https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
Martin Liška changed:
What|Removed |Added
CC||marxin at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
Andreas Krebbel changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
Jeffrey A. Law changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #12 from Vladimir Makarov ---
Author: vmakarov
Date: Thu Dec 7 17:50:54 2017
New Revision: 255471
URL: https://gcc.gnu.org/viewcvs?rev=255471=gcc=rev
Log:
2017-12-07 Vladimir Makarov
PR target/83252
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #11 from Vladimir Makarov ---
I am still working on this PR. I hope to fix it on this week or on the next
one (the patch will need a lot of testing).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #10 from Vladimir Makarov ---
Author: vmakarov
Date: Wed Nov 29 22:19:25 2017
New Revision: 255258
URL: https://gcc.gnu.org/viewcvs?rev=255258=gcc=rev
Log:
2017-11-29 Vladimir Makarov
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #9 from Vladimir Makarov ---
(In reply to Andreas Krebbel from comment #8)
> Hi Vladimir. What do you think about the additional patch?
Andreas, sorry for the delay with the answer. The patch looks reasonable for
me. If your
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
Andreas Krebbel changed:
What|Removed |Added
Priority|P3 |P2
--- Comment #8 from Andreas
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #7 from Andreas Krebbel ---
Created attachment 41699
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41699=edit
Additional changes required ontop of the proposed patch
Changes which might be required on-top of your patch.
Your
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #6 from Vladimir Makarov ---
Hi, Andreas.
Could you check the patch I attached on your bigger test case. If it works, I
will commit it.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #5 from Vladimir Makarov ---
Created attachment 41686
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41686=edit
A proposed patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #4 from Andreas Krebbel ---
Created attachment 41609
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41609=edit
Reduced testcase
gcc -O3 -march=z900 t.c
r33 is marked as unused in process_bb_lives for the compare (insn 10),
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #3 from Vladimir Makarov ---
(In reply to Andreas Krebbel from comment #0)
> Created attachment 41383 [details]
> Experimental patch
Andreas, thank you for working on the issue. You are right. The hard reg in
the clobber should be
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
--- Comment #2 from Andreas Krebbel ---
I think this is a different problem. In my case the reason for adding an
invalid rematerialization appears to be that the live ranges of an hard
registers are not correctly calculated.
I've just verified
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80818
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
CC||ktkachov at gcc dot gnu.org
15 matches
Mail list logo