[llvm] [OpenMP][Offload] Update the Logic for Configuring Auto Zero-Copy (PR #143638)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 10 20:34:24 PDT 2025
================
@@ -286,16 +286,16 @@ void PluginManager::registerLib(__tgt_bin_desc *Desc) {
}
PM->RTLsMtx.unlock();
- bool UseAutoZeroCopy = Plugins.size() > 0;
+ bool UseAutoZeroCopy = false;
auto ExclusiveDevicesAccessor = getExclusiveDevicesAccessor();
- for (const auto &Device : *ExclusiveDevicesAccessor)
- UseAutoZeroCopy &= Device->useAutoZeroCopy();
+ // APUs are homogeneous set of GPUs. Check the first device for
+ // configuring Auto Zero-Copy.
+ if (ExclusiveDevicesAccessor->size() > 0) {
----------------
carlobertolli wrote:
I think I agree with Kewen in that we do not have a case where one of the devices is an APU and the others could be something else. You could build it yourself perhaps, but it doesn't exist today as a product. If time comes when there will be such a case, we can revisit this. Reminder: MI300A is a very special case where each socket GPU devices appear as a single GPU and all sockets are identical.
https://github.com/llvm/llvm-project/pull/143638
More information about the llvm-commits
mailing list