[PATCH] D113879: [Flang] Notify conversion failure for Proc ops, types

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan added inline comments.


================
Comment at: flang/test/Fir/convert-to-llvm-invalid.fir:82
+func @unboxproc_test() {
+  %bproc = fir.call @get_bproc() : () -> (!fir.boxproc<(i32) -> ()>)
+// expected-error at +1{{failed to legalize operation 'fir.unboxproc'}}
----------------
The test is failing here.

I think it is not possible to test unboxproc and boxprochost because they need a value of tupe fir.boxproc. Since the fir.boxproc type is not convertible it is not possible to provide such a value to unboxproc or boxprochost.

Also it seems the conversion is bailing out at the first error, so it is not possible to check multiple errors.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113879