https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
Kazumoto Kojima changed:
What|Removed |Added
Attachment #33657|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #54 from Kazumoto Kojima ---
Created attachment 33657
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33657&action=edit
A possible workaround
The patch is trying to fix the result of decompose_mem_address
so as not to assign IND
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #53 from Kazumoto Kojima ---
(In reply to Oleg Endo from comment #52)
> Created attachment 33632 [details]
> Reduced case of error: in assign_by_spills, at lra-assigns.c:1335 with -m4
> -ml -O2
.ira dump of the test case has the line
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #52 from Oleg Endo ---
Created attachment 33632
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33632&action=edit
Reduced case of error: in assign_by_spills, at lra-assigns.c:1335
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #51 from Oleg Endo ---
(In reply to Oleg Endo from comment #50)
> Author: olegendo
> Date: Tue Sep 30 22:12:42 2014
> New Revision: 215744
>
> URL: https://gcc.gnu.org/viewcvs?rev=215744&root=gcc&view=rev
> Log:
> PR target/55212
> *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #50 from Oleg Endo ---
Author: olegendo
Date: Tue Sep 30 22:12:42 2014
New Revision: 215744
URL: https://gcc.gnu.org/viewcvs?rev=215744&root=gcc&view=rev
Log:
PR target/55212
* lra.c (lra_update_insn_regno_info): Handle mem args in c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #49 from Kazumoto Kojima ---
(In reply to Oleg Endo from comment #48)
> The array at i = 699 doesn't seem to contain anything valid.
It looks that
(expr_list:DF (use (mem:DF (reg/f:SI 699) [0 S8 A32]))
which shows an argument of c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #48 from Oleg Endo ---
(In reply to Oleg Endo from comment #47)
> Created attachment 33615 [details]
> reduced CSiBE /libpng-1.2.5 test
>
> I've tried compiling CSiBE (-m4 -ml). This is a stripped down pngrutil.c
> which crashes in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #47 from Oleg Endo ---
Created attachment 33615
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33615&action=edit
reduced CSiBE /libpng-1.2.5 test
I've tried compiling CSiBE (-m4 -ml). This is a stripped down pngrutil.c which
c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #46 from Oleg Endo ---
newlib 1.2.0 now builds without errors here.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #45 from Kazumoto Kojima ---
Author: kkojima
Date: Mon Sep 29 01:27:03 2014
New Revision: 215677
URL: https://gcc.gnu.org/viewcvs?rev=215677&root=gcc&view=rev
Log:
PR target/55212
* config/sh/sh.md (untyped_call): Clobber the functio
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #44 from Kazumoto Kojima ---
Author: kkojima
Date: Mon Sep 29 01:24:33 2014
New Revision: 215676
URL: https://gcc.gnu.org/viewcvs?rev=215676&root=gcc&view=rev
Log:
PR target/55212
* config/sh/sh-protos.h (sh_movsf_ie_ra_split_p): New
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #43 from Kazumoto Kojima ---
(In reply to Oleg Endo from comment #42)
> PR 54699 comes into my mind when seeing the movsf_ie patterns ... having
> another movsf_ie pattern is discomforting. But if it makes it work, so be
> it. Maybe
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #42 from Oleg Endo ---
(In reply to Kazumoto Kojima from comment #41)
> Created attachment 33601 [details]
> A trial patch for reload-loop problem
>
> My first trial is to define a special movsf_ie insn_and_split used
> when lra make
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #41 from Kazumoto Kojima ---
Created attachment 33601
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33601&action=edit
A trial patch for reload-loop problem
My first trial is to define a special movsf_ie insn_and_split used
whe
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #40 from Kazumoto Kojima ---
I've tried to see what is going on with reload loop on movsf_ie
for gcc.c-torture/compile/20050113-1.c -O1 -m4 -ml.
It looks that the problem starts at reloading
(insn 15 10 18 2 (parallel [
(
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #39 from Kazumoto Kojima ---
(In reply to Hans-Peter Nilsson from comment #38)
> Hm, you're right, but to me that indicates the patch covering-up a bug
> elsewhere than in the sh port.
Would it be better to file an independent PR for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #38 from Hans-Peter Nilsson ---
(In reply to Kazumoto Kojima from comment #34)
> For reference,
> i386.md emits a blockage (i.e. *using* and clobbering all
> registers).
> It
> looks all ports including SH emit a blockage after stori
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #37 from Oleg Endo ---
(In reply to Kazumoto Kojima from comment #36)
> I'm not sure about the old reload. LRA makes only 3 uses of r0 and it's
> relatively easy to see what is going on there with .reload comments.
> The old reload m
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #36 from Kazumoto Kojima ---
(In reply to Oleg Endo from comment #35)
> I'm just wondering ... how did/does that work without LRA (i.e. with IRA)?
I'm not sure about the old reload. LRA makes only 3 uses of r0 and it's
relatively ea
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #35 from Oleg Endo ---
(In reply to Kazumoto Kojima from comment #32)
> Created attachment 33557 [details]
> Patch for SH untyped_call
>
> * config/sh/sh.md (untyped_call): Clobber function value
> registers before call.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #34 from Kazumoto Kojima ---
(In reply to Hans-Peter Nilsson from comment #33)
> Will clobbering function-value registers before the call not cause problems
> on SH5, where function-value registers and parameters registers overlap
> (
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #33 from Hans-Peter Nilsson ---
(In reply to Kazumoto Kojima from comment #32)
> Patch for SH untyped_call
> config/sh/sh.md (untyped_call): Clobber function value registers before
> call.
(Sorry for butting in; I'm CC'ed with the pu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #32 from Kazumoto Kojima ---
Created attachment 33557
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33557&action=edit
Patch for SH untyped_call
* config/sh/sh.md (untyped_call): Clobber function value
registers before
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #31 from Kazumoto Kojima ---
Created attachment 33556
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33556&action=edit
A bit reduced test case of pr38338.c (-O0 -m4 -ml)
That case has only one basic block which looks like
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
Oleg Endo changed:
What|Removed |Added
CC||vmakarov at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #29 from Kazumoto Kojima ---
Created attachment 33528
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33528&action=edit
A trial patch
The patch is to refrain from changing to class R0_REGS for r943.
With the above 2 patches, th
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #28 from Kazumoto Kojima ---
Created attachment 33527
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33527&action=edit
Another reduced test case (with "-m4 -ml -O2 -std=gnu99")
Here is a test case for the ICE in assign_by_spill
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #27 from Kazumoto Kojima ---
Created attachment 33526
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33526&action=edit
A trial patch
It disables equiv substitution when the equiv includes some reg
assigned to a small register c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #26 from Kazumoto Kojima ---
Created attachment 33525
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33525&action=edit
.reload dump file
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #25 from Kazumoto Kojima ---
Created attachment 33524
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33524&action=edit
Reduced test case for ICE in assign_by_spill
I've looked into what is going on for the ICE in assign_by_spil
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #24 from Kazumoto Kojima ---
Author: kkojima
Date: Wed Sep 17 23:24:40 2014
New Revision: 215341
URL: https://gcc.gnu.org/viewcvs?rev=215341&root=gcc&view=rev
Log:
PR target/55212
* config/sh/predicates.md (general_movsrc_operand): T
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #23 from Oleg Endo ---
(In reply to Kazumoto Kojima from comment #22)
> Created attachment 33505 [details]
> A possible patch
>
> These last two errors could be fixed with the attached patch:
>
> * config/sh/predicates.md (gen
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #22 from Kazumoto Kojima ---
Created attachment 33505
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33505&action=edit
A possible patch
These last two errors could be fixed with the attached patch:
* config/sh/predicates.m
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #21 from Kazumoto Kojima ---
I've run check-gcc for sh-lra branch on sh4-unknown-linux-gnu
and got
=== gcc Summary ===
# of expected passes83035
# of unexpected failures169
# of unexpected successe
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #20 from Oleg Endo ---
Vlad, do you have any idea/suggestion what could be going wrong in this case?
Also, could you please have a look at the patch applied to the sh-lra branch as
r215246 and see whether it makes sense to commit it
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #19 from Oleg Endo ---
(In reply to Oleg Endo from comment #18)
> Created attachment 33489 [details]
> Reduced case for MAX_RELOAD_INSNS_NUMBER
>
> This fails when compiling with -O2 -m2e -m{l|b} with:
>
> curr_insn = (insn 713 115
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #18 from Oleg Endo ---
Created attachment 33489
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33489&action=edit
Reduced case for MAX_RELOAD_INSNS_NUMBER
This fails when compiling with -O2 -m2e -m{l|b} with:
curr_insn = (insn
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #17 from Oleg Endo ---
Author: olegendo
Date: Sat Sep 13 22:13:56 2014
New Revision: 215247
URL: https://gcc.gnu.org/viewcvs?rev=215247&root=gcc&view=rev
Log:
PR target/55212
* lra-constraints.c: Print insn when hitting MAX_RELOAD_IN
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #16 from Oleg Endo ---
(In reply to Oleg Endo from comment #5)
> Compiling the gcc.dg/atomic/c11-atomic-exec-4.c test case with '-O2 -m4 -ml
> -matomic-model=soft-gusa' results in the following:
>
This problem seems to be gone now,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #15 from Oleg Endo ---
(In reply to Oleg Endo from comment #14)
> Author: olegendo
> Date: Sat Sep 13 21:32:27 2014
> New Revision: 215246
>
> URL: https://gcc.gnu.org/viewcvs?rev=215246&root=gcc&view=rev
> Log:
> PR target/55212
> *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #14 from Oleg Endo ---
Author: olegendo
Date: Sat Sep 13 21:32:27 2014
New Revision: 215246
URL: https://gcc.gnu.org/viewcvs?rev=215246&root=gcc&view=rev
Log:
PR target/55212
* Apply patch from https://gcc.gnu.org/ml/gcc/2014-04/msg0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #13 from Oleg Endo ---
Author: olegendo
Date: Sat Sep 13 20:40:21 2014
New Revision: 215244
URL: https://gcc.gnu.org/viewcvs?rev=215244&root=gcc&view=rev
Log:
PR target/55212
* Remove LEGITIMIZE_RELOAD_ADDRESS and sh_legitimize_reloa
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #12 from Oleg Endo ---
(In reply to Oleg Endo from comment #11)
> Author: olegendo
> Date: Sat Sep 13 20:21:00 2014
> New Revision: 215243
>
> URL: https://gcc.gnu.org/viewcvs?rev=215243&root=gcc&view=rev
> Log:
> PR target/55212
> *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #11 from Oleg Endo ---
Author: olegendo
Date: Sat Sep 13 20:21:00 2014
New Revision: 215243
URL: https://gcc.gnu.org/viewcvs?rev=215243&root=gcc&view=rev
Log:
PR target/55212
* predicates.md (general_movsrc_operand, general_movdst_o
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #10 from Oleg Endo ---
(In reply to Oleg Endo from comment #9)
>
> The next failure is:
>
> gcc-sh-lra-build-sh-elf/sh-elf/libstdc++-v3/include/bits/locale_facets_nonio.
> h:1564:66: internal compiler error: in set_address_disp, at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #9 from Oleg Endo ---
(In reply to Oleg Endo from comment #8)
> Author: olegendo
> Date: Sat Sep 13 18:53:54 2014
> New Revision: 215241
>
> URL: https://gcc.gnu.org/viewcvs?rev=215241&root=gcc&view=rev
> Log:
> PR target/55212
> * p
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #8 from Oleg Endo ---
Author: olegendo
Date: Sat Sep 13 18:53:54 2014
New Revision: 215241
URL: https://gcc.gnu.org/viewcvs?rev=215241&root=gcc&view=rev
Log:
PR target/55212
* predicates.md (general_movsrc_operand, general_movdst_op
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #7 from Oleg Endo ---
An issue similar to that in c#5 has been reported here:
https://gcc.gnu.org/ml/gcc/2014-04/msg00273.html
I've tried applying that patch, but it doesn't fix the issue in c#2 nor c#5.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #6 from Oleg Endo ---
Author: olegendo
Date: Sat Sep 13 16:23:55 2014
New Revision: 215240
URL: https://gcc.gnu.org/viewcvs?rev=215240&root=gcc&view=rev
Log:
PR target/55212
* lra.c: fix comments. print insn before lra_assert.
Modi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #5 from Oleg Endo ---
Compiling the gcc.dg/atomic/c11-atomic-exec-4.c test case with '-O2 -m4 -ml
-matomic-model=soft-gusa' results in the following:
beh
(insn 207 206 208 6 (set (reg/f:SI 14 r14 [275])
(plus:SI (reg/f:SI 15
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #4 from Oleg Endo ---
(In reply to Oleg Endo from comment #2)
> However, building __divsc3 still fails for -m2 -mb, now with the
> following:
>
> beh 0 0 0
> (insn 1159 1008 1120 27 (set (reg:QI 625)
> (mem/c:QI (plus:SI (reg
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #3 from Oleg Endo ---
Created attachment 33465
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33465&action=edit
problematic libgcc divsc3 function
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
Oleg Endo changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212
--- Comment #1 from Oleg Endo ---
Author: olegendo
Date: Sun Aug 3 09:47:15 2014
New Revision: 213524
URL: https://gcc.gnu.org/viewcvs?rev=213524&root=gcc&view=rev
Log:
PR target/55212
Create branch.
Added:
branches/sh-lra/
- copied
55 matches
Mail list logo
