[llvm-commits] [PATCH] [test-suite] smallpt single source benchmark

[Evan Cheng]

Yes please do. I'm even more interested in fixing this!

Thanks,

Evan

On Dec 20, 2012, at 8:01 AM, "Redmond, Paul" <paul.redmond at intel.com> wrote:

> Hi,
> 
> Attached is a patch that adds the smallpt benchmark (http://www.kevinbeason.com/smallpt/) to the test-suite. There's lots of room for improvement given these numbers:
> 
> $ g++ -o smallpt-gcc smallpt.cpp -O3
> $ ./smallpt-gcc
> Rendering (4 spp)
> 
> real 0m5.658s
> user 0m5.632s
> sys 0m0.008s
> 
> $ icc -o smallpt-icc smallpt.cpp -O3
> $ time ./smallpt-icc
> Rendering (4 spp)
> 
> real 0m4.570s
> user 0m4.544s
> sys 0m0.012s
> 
> $ clang++ -o smallpt-clang smallpt.cpp -O3
> $ time ./smallpt-clang
> Rendering (4 spp)
> 
> real 0m6.513s
> user 0m6.488s
> sys 0m0.004s
> 
> An obvious improvement is that both ICC and GCC transforms sin and cos into sincos while LLVM does not.
> 
> Should I add this benchmark to the test-suite?
> 
> paul
> 
> <smallpt.diff>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits