[Openmp-commits] [openmp] [OpenMP][libomptarget] Use two SDMA engines (PR #73633)

Matt Arsenault via Openmp-commits openmp-commits at lists.llvm.org
Tue Nov 28 02:42:19 PST 2023


================
@@ -161,8 +161,8 @@ Error asyncMemCopy(bool UseMultipleSdmaEngines, void *Dst, hsa_agent_t DstAgent,
       Dst, DstAgent, Src, SrcAgent, Size, NumDepSignals, DepSignals,
       CompletionSignal, (hsa_amd_sdma_engine_id_t)LocalSdmaEngine,
       /*force_copy_on_sdma=*/true);
-  // Increment to use one of three SDMA engines: 0x1, 0x2, 0x4
-  LocalSdmaEngine = (LocalSdmaEngine << 1) % 7;
+  // Increment to use one of two SDMA engines: 0x1, 0x2
----------------
arsenm wrote:

Keep the comment that lists all 3? Restrict the limit to only the relevant devices?

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


More information about the Openmp-commits mailing list