This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: patch to fix constant math - third small patch
On Sat, Oct 6, 2012 at 5:55 PM, Kenneth Zadeck <zadeck@naturalbridge.com> wrote:
> This is the third patch in the series of patches to fix constant math.
> this one changes some predicates at the rtl level to use the new predicate
> CONST_SCALAR_INT_P.
> I did not include a few that were tightly intertwined with other changes.
>
> Not all of these changes are strictly mechanical. Richard, when reviewing
> this had me make additional changes to remove what he thought were latent
> bugs at the rtl level. However, it appears that the bugs were not latent.
> I do not know what is going on here but i am smart enough to not look a gift
> horse in the mouth.
>
> All of this was done on the same machine with no changes and identical
> configs. It is an x86-64 with ubuntu 12-4.
>
> ok for commit?
Patch missing, but if it's just mechanical changes and introduction
of CONST_SCALAR_INT_P consider it pre-approved.
Richard.
> in the logs below, gbBaseline is a trunk from friday and the gbWide is the
> same revision but with my patches. Some of this like gfortran.dg/pr32627 is
> obviously flutter, but the rest does not appear to be.
>
> =========
> heracles:~/gcc(13) gccBaseline/contrib/compare_tests
> gbBaseline/gcc/testsuite/gcc/gcc.log gbWide/gcc/testsuite/gcc/gcc.log
> New tests that PASS:
>
> gcc.dg/builtins-85.c scan-assembler mysnprintf
> gcc.dg/builtins-85.c scan-assembler-not __chk_fail
> gcc.dg/builtins-85.c (test for excess errors)
>
>
> heracles:~/gcc(14) gccBaseline/contrib/compare_tests
> gbBaseline/gcc/testsuite/gfortran/gfortran.log
> gbWide/gcc/testsuite/gfortran/gfortran.log
> New tests that PASS:
>
> gfortran.dg/pr32627.f03 -O3 -fomit-frame-pointer -funroll-loops (test for
> excess errors)
> gfortran.dg/pr32627.f03 -O3 -fomit-frame-pointer (test for excess errors)
> gfortran.dg/pr32627.f03 -Os (test for excess errors)
> gfortran.dg/pr32635.f -O0 execution test
> gfortran.dg/pr32635.f -O0 (test for excess errors)
> gfortran.dg/substr_6.f90 -O2 (test for excess errors)
>
> Old tests that passed, that have disappeared: (Eeek!)
>
> gfortran.dg/pr32627.f03 -O1 (test for excess errors)
> gfortran.dg/pr32627.f03 -O3 -fomit-frame-pointer -funroll-all-loops
> -finline-functions (test for excess errors)
> gfortran.dg/pr32627.f03 -O3 -g (test for excess errors)
> gfortran.dg/substring_equivalence.f90 -O (test for excess errors)
> Using /home/zadeck/gcc/gccBaseline/gcc/testsuite/config/default.exp as
> tool-and-target-specific interface file.
>
> === g++ Summary ===
>
> # of expected passes 49793
> # of expected failures 284
> # of unsupported tests 601
>
> runtest completed at Fri Oct 5 16:10:20 2012
> heracles:~/gcc(16) tail gbWide/gcc/testsuite/g++/g++.log Using
> /usr/share/dejagnu/config/unix.exp as generic interface file for target.
> Using /home/zadeck/gcc/gccWide/gcc/testsuite/config/default.exp as
> tool-and-target-specific interface file.
>
> === g++ Summary ===
>
> # of expected passes 50472
> # of expected failures 284
> # of unsupported tests 613
>
> runtest completed at Fri Oct 5 19:51:50 2012
>
>
>
>
>