[PATCH] D108317: [libomptarget][devicertl] Replace lanemask with uint64 at interface

Jon Chesterfield via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 18 12:04:51 PDT 2021


JonChesterfield created this revision.
JonChesterfield added reviewers: jdoerfert, dpalermo, grokos, tianshilei1992.
Herald added subscribers: hiraditya, tpr.
JonChesterfield requested review of this revision.
Herald added subscribers: llvm-commits, openmp-commits, cfe-commits, sstefan1.
Herald added projects: clang, OpenMP, LLVM.

Use uint64_t for lanemask on all GPU architectures at the interface
with clang. Updates tests. The deviceRTL is always linked as IR so the zext
and trunc introduced for wave32 architectures will fold after inlining.

Simplification partly motivated by amdgpu gfx10 which will be wave32, which
is awkward to express in the current arch-dependant typedef interface.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D108317

Files:
  clang/test/OpenMP/nvptx_parallel_codegen.cpp
  llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
  llvm/include/llvm/Frontend/OpenMP/OMPKinds.def
  llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
  llvm/test/Transforms/OpenMP/add_attributes.ll
  openmp/libomptarget/DeviceRTL/include/Interface.h
  openmp/libomptarget/DeviceRTL/src/Synchronization.cpp
  openmp/libomptarget/deviceRTLs/common/src/sync.cu
  openmp/libomptarget/deviceRTLs/interface.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D108317.367293.patch
Type: text/x-patch
Size: 13041 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210818/a06400b6/attachment.bin>


More information about the llvm-commits mailing list