[all-commits] [llvm/llvm-project] 81469a: [flang] use hlfir base when translating assumed-ra...

[jeanPerier via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 81469a2ee804778185e177314314f637bcb026c9
      https://github.com/llvm/llvm-project/commit/81469a2ee804778185e177314314f637bcb026c9
  Author: jeanPerier <jperier at nvidia.com>
  Date:   2024-06-10 (Mon, 10 Jun 2024)

  Changed paths:
    M flang/lib/Optimizer/Builder/HLFIRTools.cpp
    M flang/test/Lower/HLFIR/assumed-rank-inquiries-2.f90
    M flang/test/Lower/HLFIR/assumed-rank-inquiries.f90
    M flang/test/Lower/HLFIR/convert-variable-assumed-rank.f90
    M flang/test/Lower/HLFIR/select-rank.f90

  Log Message:
  -----------
  [flang] use hlfir base when translating assumed-rank entity to fir::ExtendedValue (#94822)

The hlfir::Entity to fir::ExtendedValue conversion usually uses the "fir
base" output of hlfir.declare (which is the same as the input) to avoid
introducing temporary descriptors for the sole purpose of introducing
updating lower bound information. This is possible because local lower
bounds, if any, are tracked in a vector inside the fir::ExtendedValue.

With assumed-ranks, the lower bounds cannot be tracked inside the
fir::ExtendedValue vector (their numbers is unknown at compile time).
Hence, the fir.box/fir.class used in fir::ExtendedValue in lowering must
always contain accurate local lower bound information.



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