https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45633
Jeffrey A. Law law at redhat dot com changed:
What|Removed |Added
CC||ysato at users dot
--- Comment #10 from jakub at gcc dot gnu dot org 2010-09-15 15:43 ---
Subject: Bug 45633
Author: jakub
Date: Wed Sep 15 15:42:41 2010
New Revision: 164312
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=164312
Log:
PR tree-optimization/45633
* tree-cfg.c
--- Comment #11 from jakub at gcc dot gnu dot org 2010-09-15 16:06 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #5 from jakub at gcc dot gnu dot org 2010-09-14 15:24 ---
Simplified testcase that ICEs the same way on x86_64-linux (-O3) and i686-linux
(-O3 -msse2):
int s[4];
unsigned char *t[4];
void
foo (void)
{
int i;
for (i = 0; i 4; i++)
{
s[i] = -s[i];
t[i]
--- Comment #6 from jakub at gcc dot gnu dot org 2010-09-14 16:18 ---
Created an attachment (id=21794)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21794action=view)
gcc46-pr45633-1.patch
The s[i] = -s[i]; line is also unnecessary in the testcase.
One possible patch attached,
--- Comment #7 from jakub at gcc dot gnu dot org 2010-09-14 16:20 ---
Created an attachment (id=21795)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21795action=view)
gcc46-pr45633-2.patch
Another fix, this one keeps disallowing such MINUS_EXPR. Unfortunately the
generated code
--- Comment #8 from jakub at gcc dot gnu dot org 2010-09-14 17:15 ---
I've looked at the combiner with the second patch, but I'm afraid this isn't
fixable in the combiner easily. While combiner is able for i3
(set (reg:V2DI res) (plus:V2DI (reg:V2DI temp) (mem:V2DI (symbol_ref:P t
--- Comment #9 from jakub at gcc dot gnu dot org 2010-09-14 18:43 ---
Created an attachment (id=21796)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21796action=view)
gcc46-pr45633-3.patch
Patch that uses TER to expand PLUS_EXPR/POINTER_PLUS_EXPR as MINUS_EXPR if it
is really
--- Comment #4 from jojelino at gmail dot com 2010-09-11 20:59 ---
this causes ICE
confirmed with gcc -O3 -msse2 -std=gnu99
reduced testcase
typedef unsigned char uint8_t;
typedef struct foo2
{
int offset[4];
uint8_t* ref[4];
} foo2;
int
foo(const foo2 *fooval)
{
uint8_t
--- Comment #1 from jojelino at gmail dot com 2010-09-10 11:19 ---
Created an attachment (id=21764)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21764action=view)
preprocessed source
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45633
--- Comment #2 from rguenth at gcc dot gnu dot org 2010-09-10 11:29 ---
Are you really using the fortran-dev branch?
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from jojelino at gmail dot com 2010-09-10 12:13 ---
(In reply to comment #2)
Are you really using the fortran-dev branch?
$ git branch
* trunk
it is trunk.
i don't recognize how fortran-dev version have been selected. maybe i've
mistaken.
--
12 matches
Mail list logo