https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
Jakub Jelinek changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #13 from CVS Commits ---
The master branch has been updated by Jakub Jelinek :
https://gcc.gnu.org/g:c0f772894b6b3cd8ed5c5dd09d0c7917f51cf70f
commit r11-8146-gc0f772894b6b3cd8ed5c5dd09d0c7917f51cf70f
Author: Jakub Jelinek
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #12 from Jakub Jelinek ---
I've copied the conditions on when simplify_immed_subreg is called.
CONST_DOUBLE_AS_INT_P can appear only if not TARGET_SUPPORTS_WIDE_INT,
and in that case
#define CONST_SCALAR_INT_P(X) \
(CONST_INT_P (X)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #11 from Richard Biener ---
(In reply to Jakub Jelinek from comment #10)
> Created attachment 50566 [details]
> gcc11-pr99648.patch
>
> Updated patch which doesn't ICE anymore. While I think the right thing
> would be to always
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #10 from Jakub Jelinek ---
Created attachment 50566
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50566=edit
gcc11-pr99648.patch
Updated patch which doesn't ICE anymore. While I think the right thing would
be to always punt
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #9 from Jakub Jelinek ---
(In reply to Richard Biener from comment #8)
> Hmm, yeah. I suppose we can force the constant to memory and do the subreg
> via a load?
Or just force it into a register before creating a subreg and let the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #8 from Richard Biener ---
(In reply to Jakub Jelinek from comment #7)
> --- gcc/simplify-rtx.c.jj 2021-04-09 16:18:24.275668496 +0200
> +++ gcc/simplify-rtx.c2021-04-09 19:26:24.963134240 +0200
> @@ -7059,12 +7059,19 @@
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #7 from Jakub Jelinek ---
--- gcc/simplify-rtx.c.jj 2021-04-09 16:18:24.275668496 +0200
+++ gcc/simplify-rtx.c 2021-04-09 19:26:24.963134240 +0200
@@ -7059,12 +7059,19 @@ simplify_immed_subreg (fixed_size_mode o
while
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
Jakub Jelinek changed:
What|Removed |Added
Status|WAITING |NEW
--- Comment #6 from Jakub Jelinek
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #5 from Richard Biener ---
(In reply to seurer from comment #4)
> Which RTL do you want to see?
So with a cross .expand shows we expand from
MEM [(char * {ref-all})] = 4702111234474983680;
MEM [(char * {ref-all})] = MEM
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #4 from seurer at gcc dot gnu.org ---
Which RTL do you want to see?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
--- Comment #3 from seurer at gcc dot gnu.org ---
The assembler isn't that long so here it is (from -Os):
.file "pr71522.c"
.machine power8
.section".text"
.section
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99648
Richard Biener changed:
What|Removed |Added
Keywords||wrong-code
Last reconfirmed|
13 matches
Mail list logo
