[PATCH] D111155: [fir] Add affine promotion pass

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added inline comments.


================
Comment at: flang/test/Fir/affine-promotion.fir:1
+// Test loop affine promotion
+
----------------
This comment suggests that tests for `fir.if` and `fir.load` should go to a dedicated file.


================
Comment at: flang/test/Fir/affine-promotion.fir:93
+        %cond = cmpi "sgt", %im2, %c0 : index
+        fir.if %cond {
+          %a_idx = fir.array_coor %a(%dims) %i
----------------
Would a test with `fir.if` _outside of_ `fir.do_loop` be possible? I think that it would nicely highlight  _what_ is being tested. Currently both versions of `@calc` look like tests for `fir.do_loop` specifically (that's the first thing that you notice). `fir.if`/affine.if` or `fir.load`/`affine.store` are not obvious to spot.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111155