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

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

Reply via email to