[PATCH] OpenMP Validation Suite for testing LLVM OpenMP Implementation, Patch 2 of 2

Jonathan Peyton jonathan.l.peyton at intel.com
Wed Feb 18 12:31:07 PST 2015


REPOSITORY
  rL LLVM

================
Comment at: Makefile:55
@@ +54,3 @@
+
+check-openmp: info mkdir		
+	@$(Verb) if which llvm-lit &> /dev/null; then \
----------------
Is this supposed to be a standard target like test?  I'm just curious.

================
Comment at: tools/check-openmp-test.pl:17
@@ +16,3 @@
+my $root_dir  = $ENV{ LIBOMP_WORK };
+print join('', $target_os, "_", $target_arch, "/");
+
----------------
Minor change: Use $target_platform instead of amalgamation of $target_os and $target_arch.

================
Comment at: tools/check-openmp.pl:17
@@ +16,3 @@
+my $root_dir  = $ENV{ LIBOMP_WORK };
+print join('', $root_dir, "/", "exports", "/", $target_os, "_", $target_arch, "/", "lib");
+
----------------
Minor change: Use $target_platform instead of amalgamation of $target_os and $target_arch.

http://reviews.llvm.org/D7638

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list