Skip to content

Improve the help message for an invalid calling convention#100488

Merged
bors merged 1 commit intorust-lang:masterfrom
khyperia:invalid-calling-convention-help-message
Sep 23, 2022
Merged

Improve the help message for an invalid calling convention#100488
bors merged 1 commit intorust-lang:masterfrom
khyperia:invalid-calling-convention-help-message

Conversation

@khyperia
Copy link
Copy Markdown
Contributor

@khyperia khyperia commented Aug 13, 2022

Fixes #93601

I mostly followed the suggestions of @nagisa in that issue, however, I wasn't sure how to check stability for the suggestion of "Do not suggest CCs that cannot be used due to them being unstable and feature not being enabled", so I did not implement that point.

I haven't contributed to rustc much, please feel free to point out suggestions! For example, the .map(|s| Symbol::intern(s)).collect::<Vec<_>>() seems pretty gross performance-wise, but maybe that's OK in error reporting code.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-translation Area: Translation infrastructure, and migrating existing diagnostics to SessionDiagnostic merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

The help message when an invalid calling convention is specified is getting a little unwieldy

10 participants