[PATCH] D115077: [fir] Add array operations documentation

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 7 13:34:35 PST 2021


clementval marked 2 inline comments as done.
clementval added inline comments.


================
Comment at: flang/docs/FIRArrayOperations.md:122
+
+It is only possible to use `array_fetch` on an `array_load` result value.
+
----------------
mehdi_amini wrote:
> Why this restriction?
The array "abstraction" in FIR is not meant to be used anywhere. Fortran itself doesn't have "array values" in any sane sense of that term. To avoid to introduce arrays universally in FIR we restrict the use of the abstraction to precisely where we want it and where it is useful. 



================
Comment at: flang/docs/FIRArrayOperations.md:180
+
+It is only possible to use `array_access` on an `array_load` result value.
+
----------------
mehdi_amini wrote:
> Same question: why do we need this restriction?
See answer above. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115077



More information about the llvm-commits mailing list