[Openmp-commits] [openmp] Clarify error message if TSan is missing (PR #70916)

Jonathan Schilling via Openmp-commits openmp-commits at lists.llvm.org
Wed Nov 1 02:52:34 PDT 2023


https://github.com/jons-pf created https://github.com/llvm/llvm-project/pull/70916

For an uninformed user, the error message might refer to a missing "TSan stopping operation", rather than indicating that TSan is missing **and therefore** operation is stopped.

>From ff5b38b3d986919c1f34f96d3ae024ceae6fa6df Mon Sep 17 00:00:00 2001
From: Jonathan Schilling <130992531+jons-pf at users.noreply.github.com>
Date: Wed, 1 Nov 2023 10:52:16 +0100
Subject: [PATCH] Clarify error message if TSan is missing

For an uninformed user, the error message might refer to a missing "TSan stopping operation", rather than indicating that TSan is missing **and therefore** operation is stopped.
---
 openmp/tools/archer/ompt-tsan.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openmp/tools/archer/ompt-tsan.cpp b/openmp/tools/archer/ompt-tsan.cpp
index 8b338f6b18b6e77..de77e25db2d399d 100644
--- a/openmp/tools/archer/ompt-tsan.cpp
+++ b/openmp/tools/archer/ompt-tsan.cpp
@@ -1248,7 +1248,7 @@ ompt_start_tool(unsigned int omp_version, const char *runtime_version) {
                           // tool the chance to be loaded
   {
     if (archer_flags->verbose)
-      std::cout << "Archer detected OpenMP application without TSan "
+      std::cout << "Archer detected OpenMP application without TSan; "
                    "stopping operation"
                 << std::endl;
     delete archer_flags;



More information about the Openmp-commits mailing list