[flang-commits] [PATCH] D129156: Add -fpass-plugin option to Flang

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