[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