New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NetBSD fix #197
NetBSD fix #197
Conversation
@Keno ? |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest we call this openlibm_weak_reference
(and change __strong_reference
and __warn_reference
at the same time to keep things consistent)
Co-Authored-By: Jameson Nash <vtjnash@gmail.com>
Do any of our CI services have NetBSD support? Would be nice to get a CI runner to avoid regressions. |
Nope. It might be possible on the paid tier of Cirrus (which allows you to use arbitrary GCP images) but otherwise I don't think so. |
Sorry, what I commited is not right at all... is cdefs-compat.h moving in the right direction at least? |
Please excuse my "experimental" use of this platform. I really didn't mean to abuse the PR in this way. I am not suggesting master...gufe44:netbsd-fix-openlibm_weak_reference as a solution but it should now somewhat conform to #197 (review). Can you verify this is really what you want? |
Looks right to me |
I made the new PR in #198 out of that branch. |
#196