[PATCH] D36628: [test-suite]: Adding Lulesh Proxy-app

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 15:57:53 PDT 2017


hfinkel added inline comments.


================
Comment at: MultiSource/Benchmarks/DOE-ProxyApps-C++/lulesh/lulesh-init.cc:420
+	 //pick the region
+	 regionVar = rand() % costDenominator;
+	 Index_t i = 0;
----------------
hfinkel wrote:
> This also depends on rand. See my comments on this in D36621 and D36626.
But please see my comments on a good rand implementation in D36621.


https://reviews.llvm.org/D36628





More information about the llvm-commits mailing list