Yes. The tests make sure the result is correct, but not that unrolling happens. There's another patch under review by Simon M that actually checks the assemble output to make sure the unrolling happens.
On Wed, Jun 15, 2011 at 3:15 PM, Edward Z. Yang <[email protected]> wrote: > So, uh, I ran these tests with the new codegen turned on (which don't unroll > yet), > and they all passed. Is that expected? > > Cheers, > Edward > > Excerpts from David Terei's message of Tue Jun 14 21:43:36 -0400 2011: >> Repository : ssh://darcs.haskell.org//srv/darcs/testsuite >> >> On branch : master >> >> http://hackage.haskell.org/trac/ghc/changeset/01c9b2f8ece0a7f1226d0768e811666f792787bc >> >> >--------------------------------------------------------------- >> >> commit 01c9b2f8ece0a7f1226d0768e811666f792787bc >> Author: Johan Tibell <[email protected]> >> Date: Wed Jun 8 17:27:59 2011 +0200 >> >> Add test for unrolling memcpy/memset in the x86 backend >> >> Signed-off-by: David Terei <[email protected]> >> >> tests/ghc-regress/codeGen/should_run/cgrun069.hs | 72 +++++++++++- >> .../codeGen/should_run/cgrun069_cmm.cmm | 125 >> +++++++++++++++++++- >> 2 files changed, 190 insertions(+), 7 deletions(-) >> >> >> Diff suppressed because of size. To see it, use: >> >> git show 01c9b2f8ece0a7f1226d0768e811666f792787bc >> > _______________________________________________ Cvs-ghc mailing list [email protected] http://www.haskell.org/mailman/listinfo/cvs-ghc
