[all-commits] [llvm/llvm-project] 1bd4f9: [flang][OpenMP] Put taskgroup in a new scope (#144...

[Leandro Lupori via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1bd4f9719faac77f368a7bdfdb47ead56a808375
      https://github.com/llvm/llvm-project/commit/1bd4f9719faac77f368a7bdfdb47ead56a808375
  Author: Leandro Lupori <leandro.lupori at linaro.org>
  Date:   2025-06-16 (Mon, 16 Jun 2025)

  Changed paths:
    M flang/lib/Semantics/resolve-names.cpp
    M flang/test/Lower/OpenMP/implicit-dsa.f90
    A flang/test/Lower/OpenMP/taskgroup02.f90
    M flang/test/Semantics/OpenMP/implicit-dsa.f90

  Log Message:
  -----------
  [flang][OpenMP] Put taskgroup in a new scope (#144122)

Although taskgroup is a privatizing construct, because of
task_reduction clause, a new scope was not being created for it.
This could cause an extra privatization of variables when
taskgroup was lowered, because its scope would be the same as of
the parent privatizing construct.

This fixes regressions in tests 1052_0201 and 1052_0205, from
Fujitsu testsuite.

This issue didn't happen before because implicit symbols were
being created in a different way before #142154.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications