[all-commits] [llvm/llvm-project] 23419b: [OpenMP][libarcher] Allow all possible argument se...

[Joachim via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 23419bfd1c8f26617bda47e6d4732dcbfe0c09a3
      https://github.com/llvm/llvm-project/commit/23419bfd1c8f26617bda47e6d4732dcbfe0c09a3
  Author: Joachim Protze <protze at itc.rwth-aachen.de>
  Date:   2020-10-01 (Thu, 01 Oct 2020)

  Changed paths:
    M openmp/tools/archer/ompt-tsan.cpp
    M openmp/tools/archer/tests/lit.cfg
    A openmp/tools/archer/tests/parallel/parallel-nosuppression.c
    M openmp/tools/archer/tests/parallel/parallel-simple.c

  Log Message:
  -----------
  [OpenMP][libarcher] Allow all possible argument separators in TSAN_OPTIONS

Currently, the parser used to tokenize the TSAN_OPTIONS in libomp uses
only spaces as separators, even though TSAN in compiler-rt supports
other separators like ':' or ','.
CTest uses ':' to separate sanitizer options by default.
The documentation for other sanitizers mentions ':' as separator,
but TSAN only lists spaces, which is probably where this mismatch originated.

Patch provided by  upsj

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