[Openmp-commits] [openmp] r348855 - [runtime] [test] Use more portable short options to sort(1)

Michal Gorny via Openmp-commits openmp-commits at lists.llvm.org
Tue Dec 11 07:39:26 PST 2018


Author: mgorny
Date: Tue Dec 11 07:39:26 2018
New Revision: 348855

URL: http://llvm.org/viewvc/llvm-project?rev=348855&view=rev
Log:
[runtime] [test] Use more portable short options to sort(1)

Pass `-n -s` instead of `--numeric --stable` to sort(1), as long options
are not supported by NetBSD sort implementation.  `-n` is defined
by POSIX, so it should be fully portable.  `-s` is used consistently
at least in GNU sort and FreeBSD sort, and I honestly doubt it would
cause issues with any other implementation supporting `--stable`.

Differential Revision: https://reviews.llvm.org/D55479

Modified:
    openmp/trunk/runtime/test/lit.cfg

Modified: openmp/trunk/runtime/test/lit.cfg
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/runtime/test/lit.cfg?rev=348855&r1=348854&r2=348855&view=diff
==============================================================================
--- openmp/trunk/runtime/test/lit.cfg (original)
+++ openmp/trunk/runtime/test/lit.cfg Tue Dec 11 07:39:26 2018
@@ -116,7 +116,7 @@ config.substitutions.append(("%flags", c
 
 if config.has_ompt:
     config.substitutions.append(("FileCheck", config.test_filecheck))
-    config.substitutions.append(("%sort-threads", "sort --numeric-sort --stable"))
+    config.substitutions.append(("%sort-threads", "sort -n -s"))
     if config.operating_system == 'Windows':
         # No such environment variable on Windows.
         config.substitutions.append(("%preload-tool", "true ||"))




More information about the Openmp-commits mailing list