[all-commits] [llvm/llvm-project] 303249: [flang][StackArrays] track pointers through fir.co...

[Tom Eccles via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 303249c4490a7777a744d9afd449b64ff1132a42
      https://github.com/llvm/llvm-project/commit/303249c4490a7777a744d9afd449b64ff1132a42
  Author: Tom Eccles <tom.eccles at arm.com>
  Date:   2025-01-08 (Wed, 08 Jan 2025)

  Changed paths:
    M flang/lib/Optimizer/Transforms/StackArrays.cpp
    M flang/test/Transforms/stack-arrays.fir

  Log Message:
  -----------
  [flang][StackArrays] track pointers through fir.convert (#121919)

This does add a little computational complexity because now every
freemem operation has to be tested for every allocation. This could be
improved with some more memoisation but I think it is easier to read
this way. Let me know if you would prefer me to change this to
pre-compute the normalised addresses each freemem operation is using.

Weirdly, this change resulted in a verifier failure for the fir.declare
in the previous test case. Maybe it was previously removed as dead code
and now it isn't. Anyway I fixed that too.



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