[flang-commits] [PATCH] D134285: [flang][RFC] Adding higher level FIR ops to ease expression lowering

[Roger Ferrer Ibanez via Phabricator via flang-commits]

rogfer01 added a comment.

The whole proposal seems very reasonable to me and definitely an improvement to the current status quo. Thanks a lot for putting this together @jeanPerier.



================
Comment at: flang/docs/HighLevelFIR.md:576
+
+#### Introducing the fir.apply operation
+
----------------
I'm curious about the choice of the name, why not something closer to Fortran like `array_element`? `apply` seems overly generic to me, but maybe there is precedent elsewhere? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134285