Skip to content
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.

Sign up for GitHub

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

Jump to bottom

llvm: migrate to new parameter-bearing uwtable attr #94001

Merged
merged 1 commit into from
Feb 15, 2022
Merged

llvm: migrate to new parameter-bearing uwtable attr #94001

merged 1 commit into from
Feb 15, 2022

Commits on Feb 14, 2022

  1. llvm: migrate to new parameter-bearing uwtable attr 

    In https://reviews.llvm.org/D114543 the uwtable attribute gained a flag
so that we can ask for sync uwtables instead of async, as the former are
much cheaper. The default is async, so that's what I've done here, but I
left a TODO that we might be able to do better.

While in here I went ahead and dropped support for removing uwtable
attributes in rustc: we never did it, so I didn't write the extra C++
bridge code to make it work. Maybe I should have done the same thing
with the `sync|async` parameter but we'll see.
    @durin42
    durin42 committed Feb 14, 2022
    Configuration menu
    Copy the full SHA
    0958c8f View commit details
    Browse the repository at this point in the history