[PATCH] D133966: [AMDGPU] Added __builtin_amdgcn_ds_bvh_stack_rtn

[Nuno Lopes via Phabricator via llvm-commits]

nlopes added inline comments.


================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:16910
+    llvm::Type *RetTy = ConvertType(E->getType());
+    Value *I0 = Builder.CreateInsertElement(UndefValue::get(RetTy), Rtn,
+                                            (uint64_t)0);
----------------
Please use PoisonValue here and whenever you need a placeholder value.
Thank you!


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

https://reviews.llvm.org/D133966