[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