[clang] Reapply "[Driver][ROCm][OpenMP] Fix default ockl linking for OpenMP."… (PR #126671)

Joseph Huber via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 11 06:49:41 PST 2025


================
@@ -68,7 +68,7 @@ llvm::opt::DerivedArgList *AMDGPUOpenMPToolChain::TranslateArgs(
     Action::OffloadKind DeviceOffloadKind) const {
   DerivedArgList *DAL =
       HostTC.TranslateArgs(Args, BoundArch, DeviceOffloadKind);
-  if (!DAL)
+  if (!DAL || Args.hasArg(options::OPT_fsanitize_EQ))
----------------
jhuber6 wrote:

Doesn't this leak memory? If DAL is not null we replace it with a new one.

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


More information about the cfe-commits mailing list