On Sat, Oct 6, 2012 at 5:55 PM, Kenneth Zadeck <zad...@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 > > > > >
