Anastasia added inline comments.
================
Comment at: lib/CodeGen/CGOpenCLRuntime.cpp:144
+ if (auto *I = dyn_cast<llvm::Instruction>(V)) {
+ // If the block literal is emitted as an instruction, it is an alloca
+ // and the block invoke function is stored to GEP of this alloca.
----------------
Why do we need to replace original block calls with the kernels? I think in
case of calling a block we could use the original block function and only for
enqueue use the kernel that would call the block function inside. The pointer
to the kernel wrapper could be passed as an additional parameter to
`enqueue_kernel` calls. We won't need to iterate through all IR then.
================
Comment at: lib/CodeGen/TargetInfo.cpp:8927
+llvm::Function *
+TargetCodeGenInfo::createEnqueuedBlockKernel(CodeGenFunction &CGF,
+ llvm::Function *Invoke,
----------------
Could you add some comments please?
================
Comment at: lib/CodeGen/TargetInfo.cpp:8949
+ Builder.restoreIP(IP);
+ return F;
+}
----------------
Wondering if we should add the kernel metadata (w/o args) since it was used for
long time to indicate the kernel.
================
Comment at: lib/CodeGen/TargetInfo.h:35
class Decl;
+class ASTContext;
----------------
Do we need this?
================
Comment at: test/CodeGenOpenCL/cl20-device-side-enqueue.cl:9
-// N.B. The check here only exists to set BL_GLOBAL
-// COMMON: @block_G = addrspace(1) constant void (i8 addrspace(3)*)
addrspace(4)* addrspacecast (void (i8 addrspace(3)*) addrspace(1)* bitcast ({
i32, i32, i8 addrspace(4)* } addrspace(1)*
[[BL_GLOBAL:@__block_literal_global(\.[0-9]+)?]] to void (i8 addrspace(3)*)
addrspace(1)*) to void (i8 addrspace(3)*) addrspace(4)*)
+// COMMON: %struct.__opencl_block_literal_generic = type { i32, i32, i8
addrspace(4)* }
+
----------------
Can we check generated kernel function too?
https://reviews.llvm.org/D38134
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
