[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