[Openmp-commits] [openmp] [OpenMP][OMPD] Load `libompd` with LLVM dynamic lib (PR #222455)

[Jan Patrick Lehr via Openmp-commits]

================
@@ -28,41 +32,35 @@ static void set_error(const char *msg) {
   last_error[sizeof(last_error) - 1] = '\0';
 }
 
-static void clear_error(void) {
-  last_error[0] = '\0';
-  (void)dlerror();
-}
+static void clear_error(void) { last_error[0] = '\0'; }
 
 int ompd_load_library(const char *name) {
-  const char *dlerr;
-
   clear_error();
   if (!name || !name[0]) {
     set_error("OMPD library path is empty");
-    ompd_library = NULL;
     return -1;
   }
 
-  ompd_library = dlopen(name, RTLD_LAZY);
-  dlerr = dlerror();
-  if (dlerr) {
-    set_error(dlerr);
-    ompd_library = NULL;
-    return -1;
-  }
-  if (!ompd_library) {
-    set_error("dlopen returned NULL");
+  std::string errMsg;
----------------
jplehr wrote:

```suggestion
  std::string ErrMsg;
```

Other local vars now follow this naming scheme, so make that consistent.

https://github.com/llvm/llvm-project/pull/222455