Skip to content

Commit

Permalink
Remove automatic rustfix of asm! to llvm_asm!
Browse files Browse the repository at this point in the history
This no longer works now that asm! needs an explicit import. Also, it's
been over a year since asm! landed, everyone should have transitioned by
now.
  • Loading branch information
Amanieu committed Dec 12, 2021
1 parent a0a4c7d commit b3a5537
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 71 deletions.
13 changes: 0 additions & 13 deletions compiler/rustc_builtin_macros/src/asm.rs
Original file line number Diff line number Diff line change
Expand Up @@ -42,19 +42,6 @@ fn parse_args<'a>(
ecx.struct_span_err(sp, "the legacy LLVM-style asm! syntax is no longer supported");
err.note("consider migrating to the new asm! syntax specified in RFC 2873");
err.note("alternatively, switch to llvm_asm! to keep your code working as it is");

// Find the span of the "asm!" so that we can offer an automatic suggestion
let asm_span = sp.from_inner(InnerSpan::new(0, 4));
if let Ok(s) = ecx.source_map().span_to_snippet(asm_span) {
if s == "asm!" {
err.span_suggestion(
asm_span,
"replace with",
"llvm_asm!".into(),
Applicability::MachineApplicable,
);
}
}
return Err(err);
}

Expand Down
17 changes: 0 additions & 17 deletions src/test/ui/asm/rustfix-asm.fixed

This file was deleted.

17 changes: 0 additions & 17 deletions src/test/ui/asm/rustfix-asm.rs

This file was deleted.

24 changes: 0 additions & 24 deletions src/test/ui/asm/rustfix-asm.stderr

This file was deleted.

0 comments on commit b3a5537

Please sign in to comment.