[PATCH] D94017: [flang] Fix bogus message on internal subprogram with alternate return
Pete Steinfeld via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 4 10:28:50 PST 2021
PeteSteinfeld added a comment.
In D94017#2477542 <https://reviews.llvm.org/D94017#2477542>, @klausler wrote:
> I'm confused by this change. The call checking code that's been modified pertains to absent actual arguments; but alternate returns can't be optional and must have corresponding positional actual arguments.
The code is checking for how to handle the situation where the actual argument is missing. That's potentially the case for optional arguments, but it also happens for alternate returns. So new code is saying that it's OK for the actual argument to be missing if the dummy argument is an alternate return. Formerly, we emitted an error message when calling an internal subprogram with alternate returns.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94017/new/
https://reviews.llvm.org/D94017
More information about the llvm-commits
mailing list