Skip to content

Tweak may_dangle syntax for MCP 563#107229

Closed
SoniEx2 wants to merge 1 commit into
rust-lang:masterfrom
SoniEx2:mcp563-syntax
Closed

Tweak may_dangle syntax for MCP 563#107229
SoniEx2 wants to merge 1 commit into
rust-lang:masterfrom
SoniEx2:mcp563-syntax

Conversation

@SoniEx2

@SoniEx2 SoniEx2 commented Jan 23, 2023

Copy link
Copy Markdown
Contributor

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

@rustbot

rustbot commented Jan 23, 2023

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
Contributor

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

@SoniEx2

SoniEx2 commented Jan 23, 2023

Copy link
Copy Markdown
Contributor Author

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

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

@rust-log-analyzer

Copy link
Copy Markdown
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

SoniEx2 commented Jan 23, 2023

Copy link
Copy Markdown
Contributor Author

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

@clubby789

Copy link
Copy Markdown
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
Copy Markdown
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

SoniEx2 commented Mar 14, 2023

Copy link
Copy Markdown
Contributor Author

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