Skip to content

Conversation

@tschwinge
Copy link
Member

Copy link
Member

@CohenArthur CohenArthur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Thomas! I'll open an issue to remove it once we merge the cleaned up trunk branch. Or we can keep that warning disabled if it annoys people too much :)

@CohenArthur CohenArthur merged commit df98085 into gcc-patch-dev Dec 1, 2022
@tschwinge tschwinge deleted the tschwinge/gcc-patch-dev_rust-warn-Wno-unused-parameter branch December 1, 2022 20:00
@philberty
Copy link
Member

I use no-unused-parameters only because when we have prototypes like the abstract visitors etc it seems to complain but ideally we shouldn't rely on turning it off because we miss a lot of real good warnings and we should fix those and then remove this when ready.

@CohenArthur CohenArthur added the upstream Issue regarding upstreaming gccrs into GCC label Dec 6, 2022
tschwinge added a commit that referenced this pull request Dec 14, 2022
bors bot added a commit that referenced this pull request Dec 14, 2022
1701: Standardize warning flags r=tschwinge a=tschwinge

Bring in #1670, revert #1623 and parts of #1585.


Co-authored-by: Thomas Schwinge <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

upstream Issue regarding upstreaming gccrs into GCC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants