Skip to content

Commit

Permalink
Update assembly tests
Browse files Browse the repository at this point in the history
Missed in 9c74cc3. Outside of the unwind info which changes wildly (as
expected), there is only a single functional change, namely that when
using `Option<&mut ...>` now has a single main call to `objc_msgSend,
instead of the early branch it had before that just forwarded to
`objc_msgSend`.

I'm not entirely sure why LLVM decides to do this now, I'm guessing it
has something to do with the extra unwind info possibly inhibiting the
previous optimization.

In practice, I doubt it will make much of a difference, as passing
`None` to to these is rarely what you want (you are often interested in
the out parameter).
  • Loading branch information
madsmtm committed Jan 17, 2025
1 parent 9c74cc3 commit 6866a37
Show file tree
Hide file tree
Showing 12 changed files with 5,959 additions and 1,670 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 6866a37

Please sign in to comment.