[PATCH] D129156: Add -fpass-plugin option to Flang
Rainer Orth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 24 05:54:10 PST 2022
ro added a comment.
In D129156#3949194 <https://reviews.llvm.org/D129156#3949194>, @awarzynski wrote:
>
> Sorry that you are hitting this - things like this happen, sadly. I think that the easiest to resolve it would be to tweak the expected error so that it works on Solaris as well as other popular platforms. Given that you might be the only person able to test on Solaris, would you be able to upload something for a review? Thanks!
No worries, especially given that this seems to be the first test ever to check the `Could not load library` error message.
I've just submitted a patch that works on Linux and Solaris (all that I can easily test): D138663 <https://reviews.llvm.org/D138663>.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129156/new/
https://reviews.llvm.org/D129156
More information about the cfe-commits
mailing list