[PATCH] D47644: Added backprop kernel from Rodinia Benchmark
Pankaj via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 1 11:03:12 PDT 2018
proton created this revision.
proton added reviewers: homerdin, Meinersbur.
Herald added subscribers: llvm-commits, mgorny.
Hi,
I have added the backprop kernel from Rodinia benchmark to the test-suite. I will add other kernels from Rodinia benchmark as well but before that, I want to know your views on it. I have added it to the multisource directory because there will be some kernels from Rodinia benchmark which are multisource.
More specifically I want to know if it is okay to -
-> Initializing arrays using random function (with SEED of course)
-> Print every 10th value in reference output instead of the entire array (this reduce the size of ref output and may say incorrect output as correct)
-> Use md5sum of output as ref output as it will be very small compared to output but I am not sure if it's a good idea to use md5sum here as it has collisions.
-> Can some external library like "openssl/sha.h" be used to compute checksum?
-> Is there any other library in test suite which can help in manually computing and verification of checksum?
I would like to hear any other comment/suggestion that you may have.
Repository:
rT test-suite
https://reviews.llvm.org/D47644
Files:
MultiSource/Benchmarks/CMakeLists.txt
MultiSource/Benchmarks/Rodinia/CMakeLists.txt
MultiSource/Benchmarks/Rodinia/Makefile
MultiSource/Benchmarks/Rodinia/backprop/CMakeLists.txt
MultiSource/Benchmarks/Rodinia/backprop/Makefile
MultiSource/Benchmarks/Rodinia/backprop/backprop.c
MultiSource/Benchmarks/Rodinia/backprop/backprop.h
MultiSource/Benchmarks/Rodinia/backprop/backprop.reference_output
MultiSource/Benchmarks/Rodinia/backprop/backprop_kernel.c
More information about the llvm-commits
mailing list