Re: IVOPT improvement patch

[Xinliang David Li <davidxl@google.com>]

Thanks. I will take a look.

David

On Thu, May 13, 2010 at 6:30 AM, Toon Moene <toon@moene.org> wrote:
> On 05/13/2010 03:00 PM, Toon Moene wrote:
>
>> I tried your patch today against revision 159362) and had no problem
>> patching in the diff.
>>
>> However, during stage 3 I ran into:
>>
>> /home/toon/compilers/obj-t/./prev-gcc/xgcc
>> -B/home/toon/compilers/obj-t/./prev-gcc/
>> -B/usr/snp/x86_64-unknown-linux-gnu/bin/
>> -B/usr/snp/x86_64-unknown-linux-gnu/bin/
>> -B/usr/snp/x86_64-unknown-linux-gnu/lib/ -isystem
>> /usr/snp/x86_64-unknown-linux-gnu/include -isystem
>> /usr/snp/x86_64-unknown-linux-gnu/sys-include -c -g -O2 -DIN_GCC -W
>> -Wall -Wwrite-strings -Wcast-qual -Wstrict-prototypes
>> -Wmissing-prototypes -Wmissing-format-attribute -pedantic -Wno-long-long
>> -Wno-variadic-macros -Wno-overlength-strings -Wold-style-definition
>> -Wc++-compat -DHAVE_CONFIG_H -I. -I. -I../../gcc/gcc -I../../gcc/gcc/.
>> -I../../gcc/gcc/../include -I../../gcc/gcc/../libcpp/include
>> -I../../gcc/gcc/../libdecnumber -I../../gcc/gcc/../libdecnumber/bid
>> -I../libdecnumber -DCLOOG_PPL_BACKEND -I/usr/include/libelf
>> ../../gcc/gcc/cfganal.c -o cfganal.o
>> ../../gcc/gcc/cfganal.c: In function 'find_unreachable_blocks':
>> ../../gcc/gcc/cfganal.c:281:1: internal compiler error: in
>> compute_final_value_at_exits, at tree-ssa-loop-ivopts.c:6313
>>
>> This is on x86_64-unknown-linux-gnu doing a run-of-the-mill 64-bit
>> native build.
>
> Well, that last sentence could have been more precise - this is what I did
> after applying your patch:
>
> $ cd ~/compilers/gcc && svn up && echo "`date -u` (revision `svnversion .`)"
>>> LAST_UPDATED && cd ../obj-t && rm -rf * && ../gcc/configure
> --enable-checking=release  --prefix=/usr/snp --enable-gold --enable-plugins
> --disable-multilib --disable-nls --with-arch-64=native --with-tune-64=native
> --enable-languages=fortran,c++ --enable-stage1-languages=c++
> --disable-werror && make -j8
>
> Note, for instance, the --enable-checking=release
>
> Hope this is useful,
>
> --
> Toon Moene - e-mail: toon@moene.org - phone: +31 346 214290
> Saturnushof 14, 3738 XG  Maartensdijk, The Netherlands
> At home: http://moene.org/~toon/; weather: http://moene.org/~hirlam/
> Progress of GNU Fortran: http://gcc.gnu.org/gcc-4.5/changes.html#Fortran
>