-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
New attribute parsing infrastructure #135726
Conversation
r? @Noratrieb rustbot has assigned @Noratrieb. Use |
053b11b
to
5a3ac64
Compare
This comment has been minimized.
This comment has been minimized.
5a3ac64
to
bbdb095
Compare
This comment has been minimized.
This comment has been minimized.
r? oli-obk |
bbdb095
to
5d70659
Compare
This comment has been minimized.
This comment has been minimized.
This comment was marked as resolved.
This comment was marked as resolved.
4d3437f
to
b3d6ef4
Compare
This comment has been minimized.
This comment has been minimized.
b3d6ef4
to
7319c1a
Compare
This comment has been minimized.
This comment has been minimized.
4aaff39
to
2f73b56
Compare
This comment has been minimized.
This comment has been minimized.
2f73b56
to
e8773d9
Compare
⌛ Testing commit 4daa35c with merge 7d8c6e781d347e087c7d30ea393d7dcd725ed623... |
omg :D |
@bors rollup=never p=1 bitrotty |
☀️ Test successful - checks-actions |
Finished benchmarking commit (7d8c6e7): comparison URL. Overall result: ❌✅ regressions and improvements - please read the text belowOur benchmarks found a performance regression caused by this PR. Next Steps:
@rustbot label: +perf-regression Instruction countThis is the most reliable metric that we have; it was used to determine the overall result at the top of this comment. However, even this metric can sometimes exhibit noise.
Max RSS (memory usage)Results (primary -0.1%, secondary 0.2%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesResults (primary 0.4%, secondary 0.7%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Binary sizeResults (primary -0.1%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Bootstrap: 770.447s -> 772.781s (0.30%) |
struct_span_code_err!(tcx.dcx(), item.span(), E0535, "invalid argument") | ||
struct_span_code_err!(tcx.dcx(), items[0].span(), E0535, "invalid argument") |
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.
Why? item == items[0], as destructured in let [item] = &items[..] else
Revert accidental cargo submodule update This was accidentally introduced in rust-lang#135726
Revert "Auto merge of rust-lang#135726 - jdonszelmann:attr-parsing, r=oli-obk" Trying to revert rust-lang#135726 and do some perf. runs on it. r? `@ghost`
Revert accidental cargo submodule update This was accidentally introduced in rust-lang#135726
Another step in the plan outlined in #131229
introduces infrastructure for structured parsers for attributes, as well as converting a couple of complex attributes to have such structured parsers.
This PR may prove too large to review. I left some of my own comments to guide it a little. Some general notes:
malformed repr(align(N))
#132391. I think most choices I made here are generally reasonable, but let me know if you disagree anywhere.rustc_ast_data_structures::AttributeKind
In general, a lot of the additions here are comments. I've found it very important to document new things in the 2nd commit well so other people can start using it.
Closes #132391
Closes #136717