https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #20 from Kazumoto Kojima ---
I've applied a quick fix. I'd like to keep this open for further
checks.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #19 from Kazumoto Kojima ---
Author: kkojima
Date: Tue Jan 17 04:07:51 2017
New Revision: 244516
URL: https://gcc.gnu.org/viewcvs?rev=244516=gcc=rev
Log:
PR target/78633
* config/sh/sh.md (cmpeqsi_t+1): Call copy_rtx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #18 from Jakub Jelinek ---
If the original insn that has the op.set_src rtx in it is not removed by the
splitter (which it seems it is not), and it is just expected to be removed
during DCE later, then doing copy_rtx is the right
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #17 from Oleg Endo ---
(In reply to Jakub Jelinek from comment #16)
> When a fix exists, why hasn't it been posted to gcc-patches?
Because, like I wrote in comment #13, I would like to check if there might be a
better fix for the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
Joseph S. Myers changed:
What|Removed |Added
CC||joel at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #14 from Kazumoto Kojima ---
No problem. I can continue nightly build with a minimal change
diff --git a/gcc/config/sh/sh.md b/gcc/config/sh/sh.md
index c6956a0..edadba7 100644
--- a/gcc/config/sh/sh.md
+++ b/gcc/config/sh/sh.md
@@
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #13 from Oleg Endo ---
(In reply to Kazumoto Kojima from comment #12)
> Perhaps the splitter in problem
> might have to take care of subreg case even when referencing
> a reg rtx in the input operands.
So it looks like a new rtx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #12 from Kazumoto Kojima ---
Hmm... From
https://gcc.gnu.org/onlinedocs/gccint/Sharing.html#Sharing
the above patch looks wrong. Perhaps the splitter in problem
might have to take care of subreg case even when referencing
a reg rtx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #11 from Kazumoto Kojima ---
Created attachment 40271
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40271=edit
reduce testcase
With -O1, sh4-linux compiler makes insns
(insn 67 150 165 5 (set (reg:SI 239)
(and:SI
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #10 from Kazumoto Kojima ---
Ah, don't mind. Your patch accidentally hides this PR. Now
the SH build failure comes back on trunk :-)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #9 from Dominik Vogt ---
The faulty patch has been reverted in r243256.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
Dominik Vogt changed:
What|Removed |Added
CC||vogt at linux dot vnet.ibm.com
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #7 from Oleg Endo ---
Maybe it's this thread?
https://gcc.gnu.org/ml/gcc-patches/2016-12/msg00334.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #6 from Kazumoto Kojima ---
I've tried a bi-sect on git tree. The ICE went away with
commit c95f3fa2db12f22bbb2158d18c95f6714b4292b8
Author: krebbel
Date: Fri Dec 2 08:32:40 2016
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #5 from Oleg Endo ---
(In reply to Kazumoto Kojima from comment #1)
> Here is a trial patch
>
> diff --git a/config/sh/sh.md b/config/sh/sh.md
> index c6956a0..c83bf08 100644
> --- a/config/sh/sh.md
> +++ b/config/sh/sh.md
> @@
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #4 from Kazumoto Kojima ---
Now I can't reproduce the ICEs for the revision 243217. They failed
for the revision 243091. I'm not sure whether the issue is false
positive or not. I'd like to keep this PR for a while.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
Richard Biener changed:
What|Removed |Added
Priority|P3 |P4
Target Milestone|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
--- Comment #2 from Kazumoto Kojima ---
Looks not enough. Even with the patch, I got a similar ICE for lto case
in testsuite:
FAIL: gcc.dg/atomic/stdatomic-op-1.c -O2 -flto -fuse-linker-plugin
-fno-fat-lto-objects (internal compiler error)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78633
Kazumoto Kojima changed:
What|Removed |Added
CC||olegendo at gcc dot gnu.org
---
20 matches
Mail list logo