[PATCH] D112445: [fir] Add fir.array_access op

[Pat McCormick via Phabricator via llvm-commits]

pmccormick added a comment.

The the extra documentation here would be really helpful.  In particular, from my 30,000 ft. view, it is not always clear when something is driven by a lower-level design choice tradeoff versus a goal of maintaining/capturing aspects of Fortran-level semantics that are necessary/beneficial for FIR-specific analysis and optimization.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112445/new/

https://reviews.llvm.org/D112445