[PATCH] D137154: Adding nvvm_reflect clang builtin

[Hugh Delaney via Phabricator via cfe-commits]

hdelan added inline comments.


================
Comment at: clang/include/clang/Basic/BuiltinsNVPTX.def:827
 
+BUILTIN(__nvvm_reflect, "icC*", "nc")
+
----------------
tra wrote:
> Do we actually need this patch at all.
> 
> `extern "C" int __nvvm_reflect(const char *);` appears to work just fine already:
> https://godbolt.org/z/caGenxn1e
> 
> 
It does work fine unless we are using openCL, which requires the addrspace casting as mentioned above


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137154