[flang-commits] [flang] [Flang] Add partial support for lowering procedure pointer assignment. (PR #70461)
via flang-commits
flang-commits at lists.llvm.org
Thu Nov 16 08:59:56 PST 2023
jeanPerier wrote:
> @jeanPerier In translateToExtendedValue called in genIntrinsicRefCore, procedure pointer is true for the actual.isVariable() check that is prior the actual.isProcedure() check. I am not sure if it is intentional. I added the TODO to flag procedure pointer as an actual argument to all intrinsics. Please let me know if this is reasonable.
Yes, that is intentional, a procedure pointer is a variable while a dummy procedure is not (cannot be assigned to), but some handling for procedure pointers may then be required in translateVariableToExtendedValue.
It sounds just fine to me to prevent them as arguments to all intrinsic in this patch, thanks for the update!
https://github.com/llvm/llvm-project/pull/70461
More information about the flang-commits
mailing list