[PATCH] D80804: [AMDGPU] Introduce Clang builtins to be mapped to AMDGCN atomic inc/dec intrinsics

Saiyedul Islam via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 29 10:53:48 PDT 2020


saiislam updated this revision to Diff 267295.
saiislam added a comment.

1. Updated title and description.
2. Replaced pointer element type usage to i32 type for getIntrinsic.
3. Volatile argument of the instrinsic now comes from pointer type of the first argument of the builtin.
4. Updated all test cases wrt above change.
5. Added test case for volatile pointer type.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80804

Files:
  clang/include/clang/Basic/BuiltinsAMDGPU.def
  clang/lib/CodeGen/CGBuiltin.cpp
  clang/lib/CodeGen/CodeGenFunction.h
  clang/lib/Sema/SemaChecking.cpp
  clang/test/CodeGenCXX/builtin-amdgcn-atomic-inc-dec.cpp
  clang/test/Sema/builtin-amdgcn-atomic-inc-dec-failure.cpp
  clang/test/SemaOpenCL/builtins-amdgcn-error.cl

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80804.267295.patch
Type: text/x-patch
Size: 21566 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200529/4a1f09c0/attachment-0001.bin>


More information about the cfe-commits mailing list