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.

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

Tweak may_dangle syntax for MCP 563 #107229

Closed
wants to merge 1 commit into from

Conversation

SoniEx2
Copy link
Contributor

@SoniEx2 SoniEx2 commented Jan 23, 2023

(don't mind us just using CI for development)

@rustbot
Copy link
Collaborator

rustbot commented Jan 23, 2023

r? @petrochenkov

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jan 23, 2023
@rust-log-analyzer

This comment has been minimized.

@compiler-errors
Copy link
Member

future note: It's good practice to mark as a draft and r? @ghost for PRs that are still in development

@SoniEx2
Copy link
Contributor Author

SoniEx2 commented Jan 23, 2023

oh. does CI run on drafts...? we thought it didn't?

but yes, noted about the r?, sorry >.<

@rust-log-analyzer
Copy link
Collaborator

The job x86_64-gnu-llvm-13 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)
..........................................i.ii.......................................... 14168/14233
.................................................................
failures:

---- [ui] tests/ui/dropck/mcp563-dropck-eyepatch-syntax.rs stdout ----
error: ui test compiled successfully!
status: exit status: 0
status: exit status: 0
command: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/bin/rustc" "/checkout/tests/ui/dropck/mcp563-dropck-eyepatch-syntax.rs" "-Zthreads=1" "--target=x86_64-unknown-linux-gnu" "--error-format" "json" "--json" "future-incompat" "-Ccodegen-units=1" "-Zui-testing" "-Zsimulate-remapped-rust-src-base=/rustc/FAKE_PREFIX" "-Ztranslate-remapped-path-to-local-path=no" "-Zdeduplicate-diagnostics=no" "-Cstrip=debuginfo" "--remap-path-prefix=/checkout/tests/ui=fake-test-src-base" "--emit" "metadata" "-C" "prefer-dynamic" "--out-dir" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/dropck/mcp563-dropck-eyepatch-syntax" "-A" "unused" "-Crpath" "-Cdebuginfo=0" "-Lnative=/checkout/obj/build/x86_64-unknown-linux-gnu/native/rust-test-helpers" "-L" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/dropck/mcp563-dropck-eyepatch-syntax/auxiliary"
stdout: none
stderr: none


failures:
failures:
    [ui] tests/ui/dropck/mcp563-dropck-eyepatch-syntax.rs
test result: FAILED. 14096 passed; 1 failed; 136 ignored; 0 measured; 0 filtered out; finished in 141.92s

Some tests failed in compiletest suite=ui mode=ui host=x86_64-unknown-linux-gnu target=x86_64-unknown-linux-gnu
Build completed unsuccessfully in 0:14:50

@petrochenkov petrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 23, 2023
@SoniEx2
Copy link
Contributor Author

SoniEx2 commented Jan 23, 2023

hmm, actually unsure how to proceed. gonna wait for a fix for #107231 and see what we can do from there.

@clubby789
Copy link
Contributor

This could be fixed without changing attribute validation globally, the used attribute denies other words like this:

} else if attr.has_name(sym::used) {
let inner = attr.meta_item_list();
match inner.as_deref() {
Some([item]) if item.has_name(sym::linker) => {
if !tcx.features().used_with_arg {
feature_err(
&tcx.sess.parse_sess,
sym::used_with_arg,
attr.span,
"`#[used(linker)]` is currently unstable",
)
.emit();
}
codegen_fn_attrs.flags |= CodegenFnAttrFlags::USED_LINKER;
}
Some([item]) if item.has_name(sym::compiler) => {
if !tcx.features().used_with_arg {
feature_err(
&tcx.sess.parse_sess,
sym::used_with_arg,
attr.span,
"`#[used(compiler)]` is currently unstable",
)
.emit();
}
codegen_fn_attrs.flags |= CodegenFnAttrFlags::USED;
}
Some(_) => {
tcx.sess.emit_err(ExpectedUsedSymbol { span: attr.span });
}

@SoniEx2 SoniEx2 changed the title Add MCP 563 syntax Tweak may_dangle syntax for MCP 563 Jan 24, 2023
@JohnCSimon
Copy link
Member

@SoniEx2
ping from triage - what is the status of this PR?

FYI: when a PR is ready for review, please send a message containing
@rustbot ready to switch to S-waiting-on-review so the PR is in the reviewer's backlog.

@SoniEx2 SoniEx2 marked this pull request as draft March 14, 2023 09:05
@SoniEx2
Copy link
Contributor Author

SoniEx2 commented Mar 14, 2023

we aren't entirely sure how to do this. also waiting for feedback on rust-lang/rfcs#3390 because that happened.

@SoniEx2 SoniEx2 closed this May 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants