-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Add support for registering attributes with rustc in plugins #25168
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
Conversation
This lets plugin authors opt attributes out of the `custom_attribute` and `unused_attribute` checks.
I added a test r? @huonw |
|
||
/// Register an attribute with an attribute type | ||
/// | ||
/// Registered attributes will bypass the `custom_attribute` feature gate |
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.
This and the next two lines can be one paragraph. Also, all these sentences need .
s.
(addressed) |
@@ -130,4 +135,19 @@ impl<'a> Registry<'a> { | |||
pub fn register_llvm_pass(&mut self, name: &str) { | |||
self.llvm_passes.push(name.to_owned()); | |||
} | |||
|
|||
|
|||
/// Register an attribute with an attribute type |
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.
Needs a .
(addressed) |
b1fe327
to
10e57cb
Compare
10e57cb
to
22b720a
Compare
@bors r+ |
📌 Commit 22b720a has been approved by |
This lets plugin authors opt attributes out of the `custom_attribute` and `unused_attribute` checks. cc @thepowersgang
💔 Test failed - auto-mac-32-opt |
@bors: retry |
This lets plugin authors opt attributes out of the `custom_attribute` and `unused_attribute` checks. cc @thepowersgang
This lets plugin authors opt attributes out of the
custom_attribute
and
unused_attribute
checks.cc @thepowersgang