tianshilei1992 added inline comments.
================
Comment at: clang/lib/Headers/openmp_wrappers/math.h:41
#pragma omp begin declare variant match(
\
device = {arch(nvptx, nvptx64)}, implementation = {extension(match_any)})
----------------
JonChesterfield wrote:
> @jdoerfert do you know why we have match_any here? wondering if the amdgcn
> variant below should have the same
Because `nvptx` and `nvptx64` share the same implementation. They just emit
different IRs. If AMDGCN only has one architecture, it doesn't need to use
`match_any`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104904/new/
https://reviews.llvm.org/D104904
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
