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

Joel E. Denny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 11 08:59:29 PDT 2019


jdenny added a comment.

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?


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