https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
Jakub Jelinek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
--- Comment #10 from Jakub Jelinek ---
Author: jakub
Date: Tue Mar 6 20:41:37 2018
New Revision: 258301
URL: https://gcc.gnu.org/viewcvs?rev=258301&root=gcc&view=rev
Log:
PR target/84710
* combine.c (try_combine): Use reg_or_sub
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
--- Comment #9 from Jakub Jelinek ---
(In reply to Segher Boessenkool from comment #8)
> Ah right, I had checking disabled. Need more coffee I guess.
You don't really need to build the checking compiler for this though, it is
enough to put a br
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
--- Comment #8 from Segher Boessenkool ---
Ah right, I had checking disabled. Need more coffee I guess.
The insns I see are different:
Trying 19, 20 -> 21:
19: r106:SI#0=0
20: r108:SI=zero_extend(r106:SI#0)
21: r109:SI=r108:SI 0>>0x8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
--- Comment #7 from Zdenek Sojka ---
Created attachment 43573
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43573&action=edit
original testcase
Unreduced testcase; needs additional -fkeep-inline-functions:
$ aarch64-unknown-linux-gnu-gcc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
--- Comment #6 from Zdenek Sojka ---
(In reply to Segher Boessenkool from comment #4)
> The testcase has UB of course (d is undefined).
The original testcase had defined behavior. The uninitialized use is the result
of delta/creduce reduction. T
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
--- Comment #5 from Jakub Jelinek ---
(In reply to Segher Boessenkool from comment #4)
> The testcase does not crash for me; does it need more non-default options?
>
> The testcase has UB of course (d is undefined).
The compiler needs to be con
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
--- Comment #4 from Segher Boessenkool ---
The testcase does not crash for me; does it need more non-default options?
The testcase has UB of course (d is undefined).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
--- Comment #3 from Segher Boessenkool ---
Well, yuck, I missed that part of rtl.texi (or somehow I thought this RTL would
be refused earlier).
Please use reg_or_subregno in the patch though?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
Jakub Jelinek changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84710
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
Target Milestone|---
12 matches
Mail list logo