[PATCH] D65835: [OpenMP] Permit map with DSA on combined directive

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 11 09:33:54 PDT 2019


ABataev added a comment.

In D65835#1624616 <https://reviews.llvm.org/D65835#1624616>, @jdenny wrote:

> In D65835#1624615 <https://reviews.llvm.org/D65835#1624615>, @ABataev wrote:
>
> > This is wrong. It affects all possible combinations and not only fof scalar types, all of them are affected.
>
>
> Are you saying the patch isn't sufficient because other types need to be fixed too?  Can you give an example?


Target teams private map will produce extra private in target context, other constructs either.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65835





More information about the cfe-commits mailing list