================
@@ -76,7 +77,13 @@ namespace Fortran {
namespace lower {
namespace omp {
bool requiresImplicitDefaultDeclareMapper(
- const semantics::DerivedTypeSpec &typeSpec) {
+ const semantics::DerivedTypeSpec &typeSpec,
+ bool enableImplicitDefaultMapperAllocatableMembers) {
+ // If disabled via -fno-implicit-default-mapper-allocatable-members
+ // return false, as we do not need to emit allocatable member maps.
+ if (!enableImplicitDefaultMapperAllocatableMembers)
----------------
agozillon wrote:
Last commit should I think handle both cases via number 2. I did have a little
look at 1 but I think that's definitely something for a larger follow up patch,
it'd need a bit of thinking to avoid extra maps by attaching a mapper to a
member and then having another mapper already cover that case, so effectively
doing the same map twice.
TL;DR: I think I handle the case now, I've added a few new tests to try and
verify it does what we wish.
https://github.com/llvm/llvm-project/pull/228748
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
