RKSimon added inline comments.

================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:14563
 
-    if (isa<llvm::ConstantInt>(Order)) {
-      int ord = cast<llvm::ConstantInt>(Order)->getZExtValue();
+    switch (BuiltinID) {
+    case AMDGPU::BI__builtin_amdgcn_atomic_inc32:
----------------
@saiislam @arsenm Coverity is warning that the BI__builtin_amdgcn_fence 
(fallthrough case) is not handled meaning that BuiltinAtomicOp in uninitialized


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80804

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D80804: [AMDGPU] In... Simon Pilgrim via Phabricator via cfe-commits

Reply via email to