[PATCH] D36682: [test-suite] Adding miniAMR Benchmark

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


MatzeB requested changes to this revision.
MatzeB added a comment.
This revision now requires changes to proceed.

- Integration looks ok
- The LICENSE file doesn't need an executable file mode
- I would like to hear some comments on the license statements in the files.



================
Comment at: MultiSource/Benchmarks/DOE-ProxyApps-C/miniAMR/block.h:5-7
+// Copyright (2014) 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/D36682





More information about the llvm-commits mailing list