[all-commits] [llvm/llvm-project] 37f6ba: [flang][OpenMP] Fix construct privatization in def...

[NimishMishra via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 37f6ba4fb2db2c78cda7d0a69cd0a2eff2b924e3
      https://github.com/llvm/llvm-project/commit/37f6ba4fb2db2c78cda7d0a69cd0a2eff2b924e3
  Author: NimishMishra <42909663+NimishMishra at users.noreply.github.com>
  Date:   2024-05-02 (Thu, 02 May 2024)

  Changed paths:
    M flang/include/flang/Lower/AbstractConverter.h
    M flang/lib/Lower/Bridge.cpp
    M flang/lib/Lower/OpenMP/DataSharingProcessor.cpp
    M flang/lib/Lower/OpenMP/DataSharingProcessor.h
    M flang/test/Lower/OpenMP/default-clause-byref.f90
    M flang/test/Lower/OpenMP/default-clause.f90

  Log Message:
  -----------
  [flang][OpenMP] Fix construct privatization in default clause (#72510)

Current implementation of default clause privatization incorrectly fails
to privatize in presence of non-OpenMP constructs (i.e. nested
constructs with regions whose symbols need to be privatized in the scope
of the parent OpenMP construct). This patch fixes the same by
considering non-OpenMP constructs separately by collecting symbols of a
nested region if it is a non-OpenMP construct with a region, and
privatizing it in the scope of the parent OpenMP construct.

Fixes https://github.com/llvm/llvm-project/issues/71914 and
https://github.com/llvm/llvm-project/issues/71915



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