https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
Segher Boessenkool changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
--- Comment #11 from Segher Boessenkool ---
That is a separate issue, not caused by the previous patch.
I have a patch for this, too.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
--- Comment #10 from Zdenek Sojka ---
(In reply to Segher Boessenkool from comment #8)
> Created attachment 43631 [details]
> proposed patch
>
> I cannot reproduce that exact generated code; maybe it needs tuning for some
> particular CPU?
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
--- Comment #9 from Zdenek Sojka ---
(In reply to Segher Boessenkool from comment #8)
> Created attachment 43631 [details]
> proposed patch
>
> I cannot reproduce that exact generated code; maybe it needs tuning for some
> particular CPU?
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
Segher Boessenkool changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |segher at gcc dot
gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
--- Comment #7 from Segher Boessenkool ---
I have a patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
--- Comment #6 from Segher Boessenkool ---
And the actual problem happens earlier: the earlier 63, 70 -> 71 combination
links
the much later insn 100 to 70, for cc, but there are plenty other setters and
users of cc earlier.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
--- Comment #5 from Segher Boessenkool ---
Insn 55 is a parallel, and that is split into two insns i1 and i2, both
numbered as 55. The i1 will never become part of the insn stream. It is
this insn that is deleted.
Later on insn 55 is combined
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
--- Comment #4 from ktkachov at gcc dot gnu.org ---
A carry-setting instruction gets deleted. Among the disassembly the non-failing
assembly has this:
cmp x13, 0
asr w0, w0, w4
csetw4, ne
sxtwx0,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
--- Comment #2 from ktkachov at gcc dot gnu.org ---
Fails for me with -O2 --param=tree-reassoc-width=4.
With -fno-if-conversion it doesn't fail but I don't see what the if-conversion
passes do wrong, if anything
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84780
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |8.0
14 matches
Mail list logo