Uh oh!
There was an error while loading. Please reload this page.
gh-122681: remove m_atan2()/c_atan2() helpers - #122715
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
vstinner
commented
Aug 6, 2024
Can you rebase/merge your branch to avoid the merge conflict? Then I can run buildbots on this PR. |
This comment was marked as outdated.
This comment was marked as outdated.
bedevere-bot
commented
Aug 6, 2024
skirpichev
commented
Aug 7, 2024
No failures, related to this pr. |
vstinner
left a comment
There was a problem hiding this comment.
LGTM.
@skirpichev: Did you just create to see if removing the wrapper would work or not, or do you propose to remove it?
skirpichev
commented
Aug 7, 2024
Yeah, I think we can remove it; testing with buildbots - for additional safety. But it's not urgent and I would appreciate @mdickinson blessing. Current workarounds fix only few functions in cmath. Missing bugreports - another indication, that underlying problems were fixed on affected systems. |
bedevere-bot
commented
Aug 17, 2024
🤖 New build scheduled with the buildbot fleet by @kumaraditya303 for commit 11a6f3d 🤖 If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again. |
Uh oh!
There was an error while loading. Please reload this page.