--- Comment #12 from uros at gcc dot gnu dot org 2010-03-31 18:09 ---
Subject: Bug 43528
Author: uros
Date: Wed Mar 31 18:09:04 2010
New Revision: 157880
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=157880
Log:
Backport from mainline:
2010-03-27 Uros Bizjak
--- Comment #13 from uros at gcc dot gnu dot org 2010-03-31 20:14 ---
Subject: Bug 43528
Author: uros
Date: Wed Mar 31 20:14:10 2010
New Revision: 157894
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=157894
Log:
Backport from mainline:
2010-03-27 Uros Bizjak
--- Comment #14 from ubizjak at gmail dot com 2010-03-31 20:17 ---
Fixed.
--
ubizjak at gmail dot com changed:
What|Removed |Added
Status|NEW
--- Comment #9 from ubizjak at gmail dot com 2010-03-27 12:52 ---
This patch also works:
Index: stor-layout.c
===
--- stor-layout.c (revision 157756)
+++ stor-layout.c (working copy)
@@ -1346,11 +1346,12 @@
--- Comment #10 from rguenth at gcc dot gnu dot org 2010-03-27 12:54
---
(In reply to comment #9)
This patch also works:
Index: stor-layout.c
===
--- stor-layout.c (revision 157756)
+++ stor-layout.c
--- Comment #11 from uros at gcc dot gnu dot org 2010-03-27 13:40 ---
Subject: Bug 43528
Author: uros
Date: Sat Mar 27 13:40:08 2010
New Revision: 157776
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=157776
Log:
PR tree-optimization/43528
* stor-layout.c
--- Comment #2 from ubizjak at gmail dot com 2010-03-26 09:27 ---
There is a mismatch in a guard expression. Following patch fixes ICE for me
(I'm not sure if it makes any sense, though):
Index: stor-layout.c
===
---
--- Comment #3 from rguenth at gcc dot gnu dot org 2010-03-26 10:05 ---
(In reply to comment #2)
There is a mismatch in a guard expression. Following patch fixes ICE for me
(I'm not sure if it makes any sense, though):
Index: stor-layout.c
--- Comment #4 from ebotcazou at gcc dot gnu dot org 2010-03-26 10:54
---
There is a mismatch in a guard expression. Following patch fixes ICE for me
(I'm not sure if it makes any sense, though):
No, it isn't correct, because:
/* Return 1 if T is an INTEGER_CST that can be
--- Comment #5 from ubizjak at gmail dot com 2010-03-26 11:22 ---
(In reply to comment #4)
There is a mismatch in a guard expression. Following patch fixes ICE for me
(I'm not sure if it makes any sense, though):
No, it isn't correct, because:
/* Return 1 if T is an
--- Comment #6 from ebotcazou at gcc dot gnu dot org 2010-03-26 11:50
---
If this is indeed required for tree_low_cst (which is defined as signed HWI!),
then the patch that reverses the semantics of my previous patch works as well.
Index: stor-layout.c
--- Comment #7 from rguenth at gcc dot gnu dot org 2010-03-26 12:57 ---
(In reply to comment #6)
If this is indeed required for tree_low_cst (which is defined as signed
HWI!),
then the patch that reverses the semantics of my previous patch works as
well.
Index:
--- Comment #8 from ubizjak at gmail dot com 2010-03-26 16:47 ---
(In reply to comment #7)
Yep, that's the right version.
Well, but negative sizes do not make much sense, so I think ,1 makes more
sense (with unsigned HOST_WIDE_INT then).
Eh, what now?
--
13 matches
Mail list logo