[flang-commits] [PATCH] D138140: [flang] Lower F08 FINDLOC intrinsic

[Jean Perier via Phabricator via flang-commits]

jeanPerier added a comment.

Other than the fact that this needs a rebase, LGTM, thanks.



================
Comment at: flang/test/Lower/Intrinsics/findloc.f90:21
+  findloc_test_1d = findloc(a, v)
+  ! CHECK:  %{{.*}} = fir.call @_FortranAFindloc(%[[res]], %[[arr]], %[[val]], %[[kind]], %{{.*}}, %{{.*}}, %[[mask]], %false) : (!fir.ref<!fir.box<none>>, !fir.box<none>, !fir.box<none>, i32, !fir.ref<i8>, i32, !fir.box<none>, i1) -> none
+  ! CHECK: %[[box:.*]] = fir.load %[[r]] : !fir.ref<!fir.box<!fir.heap<!fir.array<?xi32>>>>
----------------
You will need to rebase this tests after https://reviews.llvm.org/D138022 added fastmath<contract> in these calls.


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

https://reviews.llvm.org/D138140