Skip to content

Revert "[AutoDiff] Fix custom derivative thunk for Optional" #74268

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

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

MaxDesiatov
Copy link
Contributor

@MaxDesiatov MaxDesiatov commented Jun 10, 2024

Reverts #71721

Fixes preset=buildbot,tools=RA,stdlib=DA CI job, which without this revert fails on AutoDiff/SILGen/nil_coalescing.swift test.

@MaxDesiatov MaxDesiatov self-assigned this Jun 10, 2024
@MaxDesiatov
Copy link
Contributor Author

cc @asavonic @asl

@MaxDesiatov
Copy link
Contributor Author

preset=buildbot,tools=RA,stdlib=DA
@swift-ci test with preset macOS

@MaxDesiatov
Copy link
Contributor Author

@swift-ci smoke test

@MaxDesiatov MaxDesiatov added bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. AutoDiff labels Jun 10, 2024
@asl
Copy link
Contributor

asl commented Jun 10, 2024

@MaxDesiatov will you please post a link to failed CI job? This would help with triaging and fixing.

@MaxDesiatov
Copy link
Contributor Author

@asl
Copy link
Contributor

asl commented Jun 10, 2024

Ok, thanks. Looks like the test overly-optimistic matches expected code that in fact is inlined from stdlib.

@MaxDesiatov MaxDesiatov merged commit 12f402f into main Jun 11, 2024
4 checks passed
@MaxDesiatov MaxDesiatov deleted the revert-71721-optional-nil-coalescing branch June 11, 2024 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoDiff bug A deviation from expected or documented behavior. Also: expected but undesirable behavior.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants