public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/63615] New: [i686][5 Regression] FAIL: gcc.target/i386/addr-sel-1.c
@ 2014-10-22 12:39 izamyatin at gmail dot com
2014-10-22 14:10 ` [Bug target/63615] [5 " jakub at gcc dot gnu.org
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: izamyatin at gmail dot com @ 2014-10-22 12:39 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63615
Bug ID: 63615
Summary: [i686][5 Regression] FAIL:
gcc.target/i386/addr-sel-1.c
Product: gcc
Version: 5.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: izamyatin at gmail dot com
Target: i686
There is a new fail for gcc.target/i386/addr-sel-1.c on i686 after r216462
For this test we now have in asm file
movl 4(%esp), %eax
leal 1(%eax), %edx
movsbl a(%edx), %ecx
movsbl b(%edx), %eax
addl %ecx, %eax
ret
instead of earlier
movl 4(%esp), %eax
movsbl a+1(%eax), %ecx
movsbl b+1(%eax), %eax
addl %ecx, %eax
ret
Seems the change resticts some opts in postreload. Now in .postreload there are
(insn 6 21 8 2 (parallel [
(set (reg:SI 1 dx [orig:83 D.1733 ] [83])
(plus:SI (reg:SI 0 ax [96])
(const_int 1 [0x1])))
(clobber (reg:CC 17 flags))
]) ../gcc/testsuite/gcc.target/i386/addr-sel-1.c:13 220 {*addsi_1}
(nil))
(insn 8 6 10 2 (set (reg:SI 2 cx [orig:93 D.1733 ] [93])
(sign_extend:SI (mem/j:QI (plus:SI (reg:SI 1 dx [orig:83 D.1733 ]
[83])
(symbol_ref:SI ("a") <var_decl 0x7f7b4a70fc60 a>)) [0 a
S1 A8]))) ../gcc/testsuite/gcc.target/i386/addr-sel-1.c:13 148 {extendqisi2}
(nil))
while earlier there were
(insn 6 21 8 2 (parallel [
(set (reg:SI 1 dx [orig:83 D.1733 ] [83])
(plus:SI (reg:SI 0 ax [96])
(const_int 1 [0x1])))
(clobber (reg:CC 17 flags))
]) addr-sel-1-good.c:13 220 {*addsi_1}
(nil))
(insn 8 6 10 2 (set (reg:SI 2 cx [orig:93 D.1733 ] [93])
(sign_extend:SI (mem/j:QI (plus:SI (reg:SI 0 ax [96])
(const:SI (plus:SI (symbol_ref:SI ("a") <var_decl
0x7fbf3c296c60 a>)
(const_int 1 [0x1])))) [0 a S1 A8])))
addr-sel-1-good.c:13 148 {extendqisi2}
(nil))
so insn 6 is not needed
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/63615] [5 Regression] FAIL: gcc.target/i386/addr-sel-1.c
2014-10-22 12:39 [Bug target/63615] New: [i686][5 Regression] FAIL: gcc.target/i386/addr-sel-1.c izamyatin at gmail dot com
@ 2014-10-22 14:10 ` jakub at gcc dot gnu.org
2014-10-23 2:28 ` amodra at gmail dot com
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-10-22 14:10 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63615
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |amodra at gcc dot gnu.org,
| |jakub at gcc dot gnu.org
Target Milestone|--- |5.0
Summary|[i686][5 Regression] FAIL: |[5 Regression] FAIL:
|gcc.target/i386/addr-sel-1. |gcc.target/i386/addr-sel-1.
|c |c
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/63615] [5 Regression] FAIL: gcc.target/i386/addr-sel-1.c
2014-10-22 12:39 [Bug target/63615] New: [i686][5 Regression] FAIL: gcc.target/i386/addr-sel-1.c izamyatin at gmail dot com
2014-10-22 14:10 ` [Bug target/63615] [5 " jakub at gcc dot gnu.org
@ 2014-10-23 2:28 ` amodra at gmail dot com
2014-10-23 3:16 ` amodra at gmail dot com
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: amodra at gmail dot com @ 2014-10-23 2:28 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63615
Alan Modra <amodra at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed| |2014-10-23
CC|amodra at gcc dot gnu.org |amodra at gmail dot com
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
Ever confirmed|0 |1
--- Comment #1 from Alan Modra <amodra at gmail dot com> ---
Investigating
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/63615] [5 Regression] FAIL: gcc.target/i386/addr-sel-1.c
2014-10-22 12:39 [Bug target/63615] New: [i686][5 Regression] FAIL: gcc.target/i386/addr-sel-1.c izamyatin at gmail dot com
2014-10-22 14:10 ` [Bug target/63615] [5 " jakub at gcc dot gnu.org
2014-10-23 2:28 ` amodra at gmail dot com
@ 2014-10-23 3:16 ` amodra at gmail dot com
2014-10-25 7:40 ` [Bug rtl-optimization/63615] " amodra at gcc dot gnu.org
2014-10-25 10:39 ` amodra at gmail dot com
4 siblings, 0 replies; 6+ messages in thread
From: amodra at gmail dot com @ 2014-10-23 3:16 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63615
--- Comment #2 from Alan Modra <amodra at gmail dot com> ---
Breakpoint 1, simplify_plus_minus (code=code@entry=PLUS,
mode=mode@entry=SImode, op0=op0@entry=0x7ffff6d4b558,
op1=op1@entry=0x7ffff6d483a8) at /src/gcc-virgin/gcc/simplify-rtx.c:3967
(gdb) p debug_rtx(op0)
(plus:SI (reg:SI 0 ax [96])
(const_int 1 [0x1]))
$1 = void
(gdb) p debug_rtx(op1)
(symbol_ref:SI ("a") <var_decl 0x7ffff6c2f900 a>)
$2 = void
Believe it or not, this PR is due to fixing the accounting of n_constants.
Previously seeing n_constants > 1 (which is wrong, there is only one int
constant above) set "canonicalized". Now, nothing sets "canonicalized" and
simplify_plus_minus exits with NULL.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug rtl-optimization/63615] [5 Regression] FAIL: gcc.target/i386/addr-sel-1.c
2014-10-22 12:39 [Bug target/63615] New: [i686][5 Regression] FAIL: gcc.target/i386/addr-sel-1.c izamyatin at gmail dot com
` (2 preceding siblings ...)
2014-10-23 3:16 ` amodra at gmail dot com
@ 2014-10-25 7:40 ` amodra at gcc dot gnu.org
2014-10-25 10:39 ` amodra at gmail dot com
4 siblings, 0 replies; 6+ messages in thread
From: amodra at gcc dot gnu.org @ 2014-10-25 7:40 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63615
--- Comment #3 from Alan Modra <amodra at gcc dot gnu.org> ---
Author: amodra
Date: Sat Oct 25 06:45:52 2014
New Revision: 216689
URL: https://gcc.gnu.org/viewcvs?rev=216689&root=gcc&view=rev
Log:
PR rtl-optimization/63615
* simplify-rtx.c (simplify_plus_minus): Set "canonicalized" on
decomposing PLUS or MINUS if operands are not placed adjacent
in the "ops" array.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/simplify-rtx.c
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug rtl-optimization/63615] [5 Regression] FAIL: gcc.target/i386/addr-sel-1.c
2014-10-22 12:39 [Bug target/63615] New: [i686][5 Regression] FAIL: gcc.target/i386/addr-sel-1.c izamyatin at gmail dot com
` (3 preceding siblings ...)
2014-10-25 7:40 ` [Bug rtl-optimization/63615] " amodra at gcc dot gnu.org
@ 2014-10-25 10:39 ` amodra at gmail dot com
4 siblings, 0 replies; 6+ messages in thread
From: amodra at gmail dot com @ 2014-10-25 10:39 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63615
Alan Modra <amodra at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #4 from Alan Modra <amodra at gmail dot com> ---
Fixed
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-10-25 7:40 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-10-22 12:39 [Bug target/63615] New: [i686][5 Regression] FAIL: gcc.target/i386/addr-sel-1.c izamyatin at gmail dot com
2014-10-22 14:10 ` [Bug target/63615] [5 " jakub at gcc dot gnu.org
2014-10-23 2:28 ` amodra at gmail dot com
2014-10-23 3:16 ` amodra at gmail dot com
2014-10-25 7:40 ` [Bug rtl-optimization/63615] " amodra at gcc dot gnu.org
2014-10-25 10:39 ` amodra at gmail dot com
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).