[PATCH] D36680: [test-suite] Adding Pathfinder Benchmark

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 14:57:46 PDT 2017


MatzeB added a comment.

- Integration looks good to me
- I'd like to hear some clarification on the license in the file headers
- You could add a REAMDE file describing the benchmark in the benchmark directory



================
Comment at: MultiSource/Benchmarks/DOE-ProxyApps-C/Pathfinder/bitfield.h:7-9
+ * Copyright (2013) Sandia Corporation. Under the terms of Contract 
+ * DE-AC04-94AL85000 with Sandia Corporation, the U.S. Government 
+ * retains certain rights in this software.
----------------
See my comments in D36683


https://reviews.llvm.org/D36680





More information about the llvm-commits mailing list