tra added a comment.
Now that we've removed ton of @llvm.nvvm intrinsics from .td files, we have no
easy way to tell that we still do support these intrinsics (mostly for the sake
of libdevice?) by upgrading them.
Perhaps we should add the list of such intrinsics and what happens to them in a
comment section in NVPTXIntrinsics.td.
================
Comment at: llvm/include/llvm/IR/IntrinsicsNVVM.td:207
-
- def int_nvvm_brev32 : GCCBuiltin<"__nvvm_brev32">,
- Intrinsic<[llvm_i32_ty], [llvm_i32_ty], [IntrNoMem]>;
----------------
There are still remnants of __nvvm_{brev,abs,clz,...} in
include/clang/Basic/BuiltinsNVPTX.def
Now that llvm does not provide them, they should be removed from clang as well,
IMO.
https://reviews.llvm.org/D28793
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
