[Openmp-commits] [PATCH] D122764: [OpenMP][CUDA] Fix the issue that P2P memcpy doesn't work

Shilei Tian via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Jul 1 06:46:44 PDT 2022


tianshilei1992 marked an inline comment as done.
tianshilei1992 added inline comments.


================
Comment at: openmp/libomptarget/plugins/cuda/src/rtl.cpp:1091-1094
+      default:
+        REPORT("Unknown PeerAccessState %d.\n",
+               int(PeerAccessMatrix[SrcDevId][DstDevId]));
+        return OFFLOAD_FAIL;
----------------
HaohaiWen wrote:
> This default label will cause warning/error...
> error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default
Thanks for pointing that out. I'll push a fix pretty soon.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122764/new/

https://reviews.llvm.org/D122764



More information about the Openmp-commits mailing list