[PATCH] D71016: [SYCL] Implement OpenCL kernel function generation

Alexey Bader via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 14 05:31:44 PDT 2021


bader added a comment.

In D71016#3063762 <https://reviews.llvm.org/D71016#3063762>, @tschuett wrote:

> Would a codegenSYCL directory help you to separate Sema from code generation?

Moving wrapper kernel function generation to CodeGen library make sense to me.

> Doesn't this make AST non-representable of the reality,
> shouldn't the lowering happen in codegen, not in sema?

I'm not sure I understand what does "make AST non-representable of the reality" mean, but it seems to be the same suggestion as @tschuett proposed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71016



More information about the cfe-commits mailing list