================
@@ -367,6 +368,25 @@ RocmInstallationDetector::RocmInstallationDetector(
if (DetectHIPRuntime)
detectHIPRuntime();
+ if (DetectOpenMPRuntime)
+ detectOpenMPRuntime();
+}
+
+void RocmInstallationDetector::detectOpenMPRuntime() {
+ assert(OpenMPASanRTLPath.empty());
+ // Set OpenMP ASan library directory path for pre-instrumented device
+ // libraries (e.g., libompdevice.a). This path is used when linking with
+ // -fsanitize=address for OpenMP offloading.
+ OpenMPASanRTLPath = llvm::sys::path::parent_path(D.Dir);
+ llvm::sys::path::append(OpenMPASanRTLPath, "lib", "asan");
----------------
b-sumner wrote:
Yes. Both sets of libraries need to exist and this is the approach we had
agreed to on where to place the instrumented libraries.
https://github.com/llvm/llvm-project/pull/179636
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
