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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
