[all-commits] [llvm/llvm-project] 90f58e: [Flang][OpenMP] Fix loop index privatisation in Pa...

[kiranchandramohan via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 90f58eb37b30cc2f5222053dc6e7e0a187819431
      https://github.com/llvm/llvm-project/commit/90f58eb37b30cc2f5222053dc6e7e0a187819431
  Author: Kiran Chandramohan <kiran.chandramohan at arm.com>
  Date:   2023-09-01 (Fri, 01 Sep 2023)

  Changed paths:
    M flang/include/flang/Optimizer/Builder/FIRBuilder.h
    M flang/lib/Lower/Bridge.cpp
    M flang/lib/Optimizer/Builder/FIRBuilder.cpp
    A flang/test/Lower/OpenMP/hlfir-seqloop-parallel.f90

  Log Message:
  -----------
  [Flang][OpenMP] Fix loop index privatisation in Parallel region in HLFIR

HLFIR lowering always adds hlfir.declare when symbols are bound to their
address allocated on the stack. Ensure that the declare is placed along
with the alloca if it is hoisted. And always return the mlir value that
is bound to the symbol (i.e the alloca in FIR lowering and the declare
in HLFIR lowering).

Context: Loop index variables in OpenMP parallel regions should be
privatised to work correctly.

Reviewed By: tblah

Differential Revision: https://reviews.llvm.org/D158594