OK, I see the ticket, and I think that is sufficient for now. We'll fix it during the week if we can.
Is that the last of the issues on SkyNet? Note I did not yet remove the extra HAVE_NATIVE_blah's that you found in my Toom code. But I think it is safer, given that so much testing has already happened, that I also make a ticket for that and we'll fix that in a service release. I'll make an MPIR 1.2.1 target in trac and assign it to that as a blocker. Bill. 2009/5/31 Jason Moxham <[email protected]>: > > The error is also in mpir-1.1 on mark with > export CC=cc && ./configure --build=none && make -j 2 && make -j 2 check > > > On Sunday 31 May 2009 13:33:24 Bill Hart wrote: >> Unfortunately they are the only sun machines with that architecture we >> have access to. >> >> At any rate, let's try mpir-1.1. If it is there, I guess we make a >> ticket and fix it later this week, if it isn't a compiler bug. I'm >> sure it is harmless given that the tests pass under an ordinary build. >> >> Bill. >> >> 2009/5/31 Jason Moxham <[email protected]>: >> > If the machines were faster I would , I suspect it the error is when we >> > configure with build=none and only in mpn_divrem_euclidean_q?_1.c when we >> > build with Sun's cc >> > >> > On Sunday 31 May 2009 13:24:01 Bill Hart wrote: >> >> That seems wise. In fact, I wonder if this error was in the original >> >> GMP. It might be worth running the same test for GMP. At least we >> >> could exclude that this has been there all along. >> >> >> >> Bill. >> >> >> >> 2009/5/31 Jason Moxham <[email protected]>: >> >> > This error should also be in mpir-1.1 >> >> > I'll run a test for it on mark , I'll use mark2 for my testing , they >> >> > are so SLOW >> >> > >> >> > On Sunday 31 May 2009 13:07:09 Bill Hart wrote: >> >> >> If you are *sure* it is a miscompilation, let's make a ticket for it. >> >> >> We also need to understand if this can ever affect a real user, >> >> >> assuming we had users who like to use the Sun compiler, or is this >> >> >> something that can only affect them if they select build=none, which >> >> >> I doubt a real user would do. >> >> >> >> >> >> If only the latter, then this needn't block an alpha release, though >> >> >> we should fix it over the next week before final or in a service >> >> >> release. >> >> >> >> >> >> Bill. >> >> >> >> >> >> 2009/5/31 Jason Moxham <[email protected]>: >> >> >> > The file is divrem_euclidean_qr_1 , and it looks like cc is >> >> >> > compiling it wrong when we select --build=none >> >> >> > >> >> >> > it passed with these configure options except the last with cc and >> >> >> > CC #standard build >> >> >> > build $C $CX "" >> >> >> > # all features >> >> >> > build $C $CX "--enable-cxx --enable-gmpcompat" >> >> >> > # all features and debug >> >> >> > build $C >> >> >> > $CX "--enable-cxx --enable-gmpcompat --enable-assert >> >> >> > --enable-alloca=debug" # maximum debug >> >> >> > build $C >> >> >> > $CX "--enable-cxx --enable-gmpcompat --enable-assert >> >> >> > --enable-alloca=debug --build=none" >> >> >> > >> >> >> > and it passes everthing with gcc and gcc-4.3.3 >> >> >> > >> >> >> > I just corrected an error that would of kicked in at 2^31 limbs in >> >> >> > divrem_euclidean_1.c >> >> >> > >> >> >> > On Sunday 31 May 2009 12:41:30 Jason Moxham wrote: >> >> >> >> using cc and CC on mark or mark2 >> >> >> >> configure --enable-cxx --enable-gmpcompat --enable-assert >> >> >> >> --enable-alloca=debug --build=none >> >> >> >> >> >> >> >> make[4]: Entering directory `/tmp/mark2/tests/mpz' >> >> >> >> PASS: t-addsub >> >> >> >> PASS: t-cmp >> >> >> >> PASS: t-mul >> >> >> >> PASS: t-mul_i >> >> >> >> PASS: t-tdiv >> >> >> >> ERROR: remainders from mpz_tdiv_qr_ui and mpz_tdiv_r_ui differ >> >> >> >> dividend = 4B3 >> >> >> >> divisor = FFE00078 >> >> >> >> /bin/bash: line 4: 5999 Abort (core dumped) >> >> >> >> ${dir}$tst FAIL: t-tdiv_ui >> >> >> >> PASS: t-fdiv >> >> >> >> ERROR: remainders from mpz_fdiv_qr_ui and mpz_fdiv_r_ui differ >> >> >> >> dividend = 4B3 >> >> >> >> divisor = FFE00078 >> >> >> >> /bin/bash: line 4: 6007 Abort (core dumped) >> >> >> >> ${dir}$tst FAIL: t-fdiv_ui >> >> >> >> ERROR: remainders from mpz_cdiv_qr_ui and mpz_cdiv_r_ui differ >> >> >> >> dividend = 4B3 >> >> >> >> divisor = FFE00078 >> >> >> >> /bin/bash: line 4: 6012 Abort (core dumped) >> >> >> >> ${dir}$tst FAIL: t-cdiv_ui >> >> >> >> PASS: t-gcd >> >> >> >> PASS: t-gcd_ui >> >> >> >> PASS: t-lcm >> >> >> >> PASS: dive >> >> >> >> PASS: dive_ui >> >> >> >> PASS: t-sqrtrem >> >> >> >> PASS: convert >> >> >> >> PASS: io >> >> >> >> PASS: t-inp_str >> >> >> >> PASS: logic >> >> >> >> PASS: bit >> >> >> >> PASS: t-powm >> >> >> >> PASS: t-powm_ui >> >> >> >> PASS: t-pow >> >> >> >> PASS: t-div_2exp >> >> >> >> PASS: reuse >> >> >> >> PASS: t-root >> >> >> >> PASS: t-perfsqr >> >> >> >> PASS: t-jac >> >> >> >> PASS: t-bin >> >> >> >> PASS: t-get_d >> >> >> >> PASS: t-get_d_2exp >> >> >> >> PASS: t-get_si >> >> >> >> PASS: t-set_d >> >> >> >> PASS: t-set_si >> >> >> >> PASS: t-fac_ui >> >> >> >> PASS: t-fib_ui >> >> >> >> PASS: t-lucnum_ui >> >> >> >> PASS: t-scan >> >> >> >> PASS: t-fits >> >> >> >> PASS: t-divis >> >> >> >> PASS: t-divis_2exp >> >> >> >> PASS: t-cong >> >> >> >> PASS: t-cong_2exp >> >> >> >> PASS: t-sizeinbase >> >> >> >> PASS: t-set_str >> >> >> >> PASS: t-aorsmul >> >> >> >> PASS: t-cmp_d >> >> >> >> PASS: t-cmp_si >> >> >> >> PASS: t-hamdist >> >> >> >> PASS: t-oddeven >> >> >> >> PASS: t-popcount >> >> >> >> PASS: t-set_f >> >> >> >> PASS: t-io_raw >> >> >> >> PASS: t-import >> >> >> >> PASS: t-export >> >> >> >> PASS: t-pprime_p >> >> >> >> PASS: t-perfpow >> >> >> >> ============================================================= >> >> >> >> 3 of 57 tests failed >> >> >> >> >> >> >> >> >> >> >> >> This looks like a compiler error , or a real one >> >> > > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "mpir-devel" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/mpir-devel?hl=en -~----------~----~----~----~------~----~------~--~---
