[all-commits] [llvm/llvm-project] bacbf2: [flang] allow assumed-rank box in fir.alloca (#95947)

[jeanPerier via All-commits]

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

  Changed paths:
    M flang/lib/Optimizer/Dialect/FIROps.cpp
    M flang/test/Fir/convert-to-llvm.fir

  Log Message:
  -----------
  [flang] allow assumed-rank box in fir.alloca (#95947)

The alloca can be maximized with the maximum number or ranks, which is
reasonable (15 currently as per the standard). Introducing a rank based
dynamic allocation would complexify alloca hoisting and stack size
analysis (this can be revisited if the standard changes to allow more
ranks).

No change is needed since this is already reflected in how the fir.box
type is translated to LLVM.



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