This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix PR/46200 -- ivopt bug in test condition cost computation
- From: Xinliang David Li <davidxl at google dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Zdenek Dvorak <rakdver at kam dot mff dot cuni dot cz>
- Date: Sat, 6 Nov 2010 01:04:17 -0700
- Subject: Re: Fix PR/46200 -- ivopt bug in test condition cost computation
- References: <AANLkTi=TiOqvQsCGUwAP+BdYuNKoXkMR+bUPAJ3LP=8d@mail.gmail.com> <AANLkTikD+BnBOBxS3vAJA4LM0DYM2gYynsXYQW22pVc0@mail.gmail.com>
Sorry for the late reply -- reproduced. It is likely some bug
triggered by this change. I will take a look.
David
On Fri, Nov 5, 2010 at 1:32 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Fri, Oct 29, 2010 at 1:28 PM, Xinliang David Li <davidxl@google.com> wrote:
>> Hi, please review the patch attached. Regression and some performance
>> test is under going.
>>
>> Thanks,
>>
>> David
>>
>>
>> 2010-10-29 ?Xinliang David Li ?<davidxl@google.com>
>>
>> ? ? ? ?PR target/46200
>> ? ? ? ?* tree-ssa-loop-ivopts.c (get_computation_cost_at):
>> ? ? ? ?Adjust cbase if the use stmt is after iv update.
>>
>
> This patch may have caused:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46316
>
> You need to build a 32bit gcc to reproduce it. I am using
>
> CC="gcc -m32" CXX="g++ -m32" ?configure ?i686-linux
>
> on Fedora 16/Intel64 to build 32bit GCC.
>
>
> --
> H.J.
>