[PATCH] D137875: clang/AMDGPU: Drop volatile from builtin_atomic* pointers

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 11 16:19:25 PST 2022


arsenm created this revision.
arsenm added a reviewer: yaxunl.
Herald added subscribers: kosarev, kerbowa, tpr, dstuttard, jvesely, kzhuravl.
Herald added a project: All.
arsenm requested review of this revision.
Herald added a subscriber: wdng.

The codegen takes the volatileness from the used value,
so this didn't really matter.


https://reviews.llvm.org/D137875

Files:
  clang/include/clang/Basic/BuiltinsAMDGPU.def


Index: clang/include/clang/Basic/BuiltinsAMDGPU.def
===================================================================
--- clang/include/clang/Basic/BuiltinsAMDGPU.def
+++ clang/include/clang/Basic/BuiltinsAMDGPU.def
@@ -75,11 +75,11 @@
 BUILTIN(__builtin_amdgcn_fence, "vUicC*", "n")
 BUILTIN(__builtin_amdgcn_groupstaticsize, "Ui", "n")
 
-BUILTIN(__builtin_amdgcn_atomic_inc32, "UZiUZiD*UZiUicC*", "n")
-BUILTIN(__builtin_amdgcn_atomic_inc64, "UWiUWiD*UWiUicC*", "n")
+BUILTIN(__builtin_amdgcn_atomic_inc32, "UZiUZi*UZiUicC*", "n")
+BUILTIN(__builtin_amdgcn_atomic_inc64, "UWiUWi*UWiUicC*", "n")
 
-BUILTIN(__builtin_amdgcn_atomic_dec32, "UZiUZiD*UZiUicC*", "n")
-BUILTIN(__builtin_amdgcn_atomic_dec64, "UWiUWiD*UWiUicC*", "n")
+BUILTIN(__builtin_amdgcn_atomic_dec32, "UZiUZi*UZiUicC*", "n")
+BUILTIN(__builtin_amdgcn_atomic_dec64, "UWiUWi*UWiUicC*", "n")
 
 // FIXME: Need to disallow constant address space.
 BUILTIN(__builtin_amdgcn_div_scale, "dddbb*", "n")


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D137875.474886.patch
Type: text/x-patch
Size: 966 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221112/4074c324/attachment.bin>


More information about the cfe-commits mailing list