--- Comment #17 from ebotcazou at gcc dot gnu dot org 2006-01-26 07:04
---
http://gcc.gnu.org/ml/gcc-patches/2006-01/msg01786.html
--
ebotcazou at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #16 from sayle at gcc dot gnu dot org 2006-01-26 02:48 ---
Subject: Bug 25703
Author: sayle
Date: Thu Jan 26 02:48:01 2006
New Revision: 110242
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110242
Log:
PR rtl-optimization/25703
* combine.c (try_comb
--- Comment #15 from ebotcazou at gcc dot gnu dot org 2006-01-25 20:54
---
> LAST_UPDATED: Wed Jan 25 17:12:12 UTC 2006 (revision 110216)
>
> === acats tests ===
> FAIL: c32001e
> FAIL: c64105b
> FAIL: c95086b
> FAIL: cdd2a02
>
> === acats Summa
--- Comment #14 from hjl at lucon dot org 2006-01-25 20:47 ---
FYI, I got
LAST_UPDATED: Wed Jan 25 17:12:12 UTC 2006 (revision 110216)
=== acats tests ===
FAIL: c32001e
FAIL: c64105b
FAIL: c95086b
FAIL: cdd2a02
=== acats Summary ===
# of expecte
--- Comment #13 from ebotcazou at gcc dot gnu dot org 2006-01-25 20:15
---
> My apologies once again for the inconvenience. In the previous version of the
> patch I'd mistakenly assumed that STRICT_LOW_PART was some indication that the
> SUBREG only affected the "low_part". Investigat
--- Comment #12 from hjl at lucon dot org 2006-01-25 20:14 ---
Patch in comment 11 looks good. c34006a, cc1226b and Jan's C testcase now
pass.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25703
--- Comment #11 from roger at eyesopen dot com 2006-01-25 19:52 ---
Created an attachment (id=10729)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10729&action=view)
patch v2
Here's a revised version of the patch that also handles the STRICT_LOW_PART
case.
My apologies once again
--- Comment #10 from ebotcazou at gcc dot gnu dot org 2006-01-25 18:59
---
> No, it still needs -mtune=pentiumpro to pass.
OK, thanks, I see the problem.
Roger, if you haven't already plugged it, the very same hole is present in the
STRICT_LOW_PART case too. For Jan's testcase:
(ins
--- Comment #9 from hjl at lucon dot org 2006-01-25 18:41 ---
No, it still needs -mtune=pentiumpro to pass.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25703
--- Comment #8 from ebotcazou at gcc dot gnu dot org 2006-01-25 18:27
---
> I tried the patch in comment 5. It doesn't fix c34006a and cc1226b on
> Linux/i686.
Does it change anything for Jan's testcase?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25703
--- Comment #7 from hjl at lucon dot org 2006-01-25 18:19 ---
I tried the patch in comment 5. It doesn't fix c34006a and cc1226b on
Linux/i686.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25703
--- Comment #6 from ebotcazou at gcc dot gnu dot org 2006-01-25 13:52
---
*** Bug 25899 has been marked as a duplicate of this bug. ***
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25703
--- Comment #5 from roger at eyesopen dot com 2006-01-25 01:05 ---
I'm testing the following patch...
Index: combine.c
===
*** combine.c (revision 109912)
--- combine.c (working copy)
*** try_combine (rtx i3
--- Comment #4 from ebotcazou at gcc dot gnu dot org 2006-01-24 21:37
---
Recategorizing.
--
ebotcazou at gcc dot gnu dot org changed:
What|Removed |Added
Compo
14 matches
Mail list logo
