[PATCH] D102107: [OpenMP] Codegen aggregate for outlined function captures

Giorgis Georgakoudis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 15 14:52:43 PDT 2021


ggeorgakoudis added a comment.

In D102107#2819869 <https://reviews.llvm.org/D102107#2819869>, @lebedev.ri wrote:

> (This is not offload-specific, right?)
> This does not bring any compatibility issues, right?

It's not offload-specific, so the patch applies to host OpenMP too. The patch, as is, does not introduce any incompatibilities, the OpenMP runtime parallel call interface stays the same and the only change is the aggregation of the outlined function arguments in a struct. However, it makes sense to simplify the OpenMP parallel call runtime interface to avoid variadic arguments. This change is not necessary for this patch, but I'm considering to include it too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102107



More information about the cfe-commits mailing list