================
@@ -68,7 +68,9 @@ llvm::opt::DerivedArgList
*AMDGPUOpenMPToolChain::TranslateArgs(
Action::OffloadKind DeviceOffloadKind) const {
DerivedArgList *DAL =
HostTC.TranslateArgs(Args, BoundArch, DeviceOffloadKind);
- if (!DAL || Args.hasArg(options::OPT_fsanitize_EQ))
+ // Skip sanitize options passed from the HostTC. The decision to instrument
+ // device code is computed only by 'shouldSkipSanitizeOption'.
+ if (!DAL && DAL->hasArg(options::OPT_fsanitize_EQ))
----------------
ampandey-1995 wrote:
> if it's not legal for some case, then emit an error. If the user doesn't want
> it then they can do `-Xarch-host -fsanitize=address`.
We actually emit the warning not error, since it's legal for the host to be
asan instrumented but for device it depends based on ```--offload-arch=```
containing :xnack+. If :xnack+ then allow it otherwise emit warning silently
skip it.
https://github.com/llvm/llvm-project/pull/126671
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
