[PATCH] D47644: Added backprop kernel from Rodinia Benchmark

Brian Homerding via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 1 13:47:06 PDT 2018


homerdin added a comment.

Hello,

We can't rely on the stdlib rand() when verifying.  You can include an implementation to use (eg `MultiSource/Benchmarks/DOE-ProxyApps-C/miniAMR/glibc_compat_rand`) so the output can be verified.

I'm unsure about a checksum lib in the test-suite, but the reference_output should be smaller.

Make sure to add [test-suite] to the title so that it is easier to find/filter for other interested community members, also you need to include Rodinia's license and mention it in `test-suite/LICENSE.TXT`

Thanks


Repository:
  rT test-suite

https://reviews.llvm.org/D47644





More information about the llvm-commits mailing list