lildmh marked an inline comment as done.
lildmh added inline comments.
================
Comment at: lib/Sema/SemaOpenMP.cpp:14805
+ if (!Type->isStructureOrClassType() && !Type->isUnionType() &&
+ (MapperIdScopeSpec.isSet() || MapperId.getAsString() != "default")) {
+ SemaRef.Diag(Loc, diag::err_omp_mapper_wrong_type);
----------------
ABataev wrote:
> Why need an additional check for scope and not "default" id? I don't see this
> additional requirement in the standard.
It's because every variable in map clauses will check this, including those are
not struct, class, or union.
Using this, e.g., mapping an integer won't report error that it doesn't have a
mapper.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67978/new/
https://reviews.llvm.org/D67978
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
