dfukalov marked an inline comment as done.
dfukalov added inline comments.

================
Comment at: lib/CodeGen/CGBuiltin.cpp:9866
+  case AMDGPU::BI__builtin_amdgcn_ds_fmax: {
+    llvm::SmallVector<llvm::Value *, 5> Args;
+    for (unsigned I = 0; I != 5; ++I)
----------------
b-sumner wrote:
> Can the pointer argument address space be checked here?
added code to check pointer and flaot parameters
though this part is not active at the moment since the builtins are not 
processed as custom.
going to switch to custom path by removing attribute of the intrinsics in llvm 
trunk in next step


https://reviews.llvm.org/D43281



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to