[all-commits] [llvm/llvm-project] fb2099: [flang][OpenMP] Set isNewBlock directly on OpenMP ...

[Krzysztof Parzyszek via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fb209929e1b1abbbebae4f3fdbd080c8e54023a1
      https://github.com/llvm/llvm-project/commit/fb209929e1b1abbbebae4f3fdbd080c8e54023a1
  Author: Krzysztof Parzyszek <Krzysztof.Parzyszek at amd.com>
  Date:   2025-06-23 (Mon, 23 Jun 2025)

  Changed paths:
    M flang/include/flang/Lower/PFTBuilder.h
    M flang/lib/Lower/PFTBuilder.cpp
    A flang/test/Lower/OpenMP/multiple-entry-points.f90

  Log Message:
  -----------
  [flang][OpenMP] Set isNewBlock directly on OpenMP constructs (#144593)

When the PFT builder decides that an evaluation needs a new block it
checks if the evaluation has nested evaluations. In such case it sets
the flag on the first nested evaluation. This works under the assuption
that such an evaluation only serves as a container, and does not, by
itself, generate any code.

This fails for OpenMP constructs that contain nested evaluations because
the top-level evaluation does generate code that wraps the code from the
nested evaluations. In such cases, the code for the top-level evaluation
may be emitted in a wrong place.

When setting the `isNewBlock` flag, recognize OpenMP directives, and
treat them accordingly.

This fixes https://github.com/llvm/llvm-project/issues/139071



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