[PATCH] D49883: [test-suite] Added Image Processing Kernels Using Benchmark Library: Dilate Kernel

[Pankaj via Phabricator via llvm-commits]

proton added inline comments.


================
Comment at: MicroBenchmarks/ImageProcessing/Dilate/main.cpp:79
+
+  dilateKernel(inputImage, outputImage, height, width, temp);
+
----------------
this call is made to warm up the cache


================
Comment at: MicroBenchmarks/ImageProcessing/Dilate/main.cpp:86
+  if (state.range(0) == 20) {
+    saveImage(outputImage, (const char *)"testFailed.txt", height, width);
+  }
----------------
Using the output image so that the "dilateKernel" calls above are not optimized out. This is is always false thought because we never pass state.range(0) as 20.


Repository:
  rT test-suite

https://reviews.llvm.org/D49883