https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #6 from Jakub Jelinek ---
Created attachment 40204
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40204&action=edit
gcc7-pr78614.patch
This fixes the testcase for me in a cross. Could somebody try
--enable-checking=yes,rtl boo
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #7 from Jakub Jelinek ---
Created attachment 40205
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40205&action=edit
gcc7-pr78614-2.patch
Perhaps better patch (if it works). The previous patch would copy even rtxes
that simplif
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #8 from Markus Trippelsdorf ---
(In reply to Jakub Jelinek from comment #7)
> Created attachment 40205 [details]
> gcc7-pr78614-2.patch
>
> Perhaps better patch (if it works). The previous patch would copy even
> rtxes that simplify
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #9 from Jakub Jelinek ---
Created attachment 40206
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40206&action=edit
gcc7-pr78614-3.patch
The better patch actually doesn't seem to be much better, because
simplify_replace_fn when
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #10 from Markus Trippelsdorf ---
(In reply to Jakub Jelinek from comment #9)
> Created attachment 40206 [details]
> gcc7-pr78614-3.patch
>
> The better patch actually doesn't seem to be much better, because
> simplify_replace_fn when
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
Jakub Jelinek changed:
What|Removed |Added
Attachment #40206|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #12 fro
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #13 from Jakub Jelinek ---
By last patch you mean #c7? #c11 is just an improvement on top of that.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #14 from Alan Modra ---
I meant the last one attached, which I see is the same as the first one. ie.
just use copy_rtx and don't worry about extra rtl.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #15 from Jakub Jelinek ---
Oops, I've attached the wrong one.
The third one was meant to be the
http://gcc.gnu.org/ml/gcc-patches/2016-11/msg03116.html
one.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #16 from Alan Modra ---
The patch referenced in comment #15 alone, ie. without
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg03117.html, results in an
--enable-checking=yes,rtl bootstrap failure on powerpc64le-linux.
/home/amodra/src
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #17 from Alan Modra ---
I see the same sort of invalid rtl sharing errors on powerpc64le with both
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg03117.html and
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg03116.html applied.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #18 from Alan Modra ---
Ditto for just https://gcc.gnu.org/ml/gcc-patches/2016-11/msg03117.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #19 from Alan Modra ---
Cancel those last three comments. I wasn't actually testing with Jakub's
rs6000.c patch.
Everything now looks good with Jakub's rs6000.c and simplify-rtx.c patch, plus
the rs6000.c insn_is_swappable_p patch I
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #20 from Jakub Jelinek ---
(In reply to Alan Modra from comment #19)
> Cancel those last three comments. I wasn't actually testing with Jakub's
> rs6000.c patch.
>
> Everything now looks good with Jakub's rs6000.c and simplify-rtx.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #21 from Jakub Jelinek ---
Author: jakub
Date: Thu Dec 1 07:56:49 2016
New Revision: 243099
URL: https://gcc.gnu.org/viewcvs?rev=243099&root=gcc&view=rev
Log:
PR target/78614
* config/rs6000/rs6000.c (rs6000_frame_re
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
Jakub Jelinek changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #23 from Bill Schmidt ---
I've backported the insn_is_swappable_p bugfix to gcc-5-branch and
gcc-6-branch. Thanks again for fixing that, Alan!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #24 from Jakub Jelinek ---
Author: jakub
Date: Fri Dec 2 15:42:04 2016
New Revision: 243194
URL: https://gcc.gnu.org/viewcvs?rev=243194&root=gcc&view=rev
Log:
PR target/78614
* rtl.c (copy_rtx): Don't clear used flag
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
Daniel Black changed:
What|Removed |Added
CC||daniel.black at au dot ibm.com
--- Commen
20 matches
Mail list logo
