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

Allow suboptimal_flops in const functions #8009

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

xFrednet
Copy link
Member

@xFrednet xFrednet commented Nov 20, 2021

This PR allows clippy::suboptimal_flops in constant functions. The check also effects the clippy::imprecise_flops lint logic. However, this doesn't have any effects as all functions checked for are not const and can therefore not be found in such functions.


changelog: [suboptimal_flops]: No longer triggers in constant functions

Closes: #8004

@rust-highfive
Copy link

r? @giraffate

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 20, 2021
@xFrednet xFrednet force-pushed the 8004-suboptimal-flops-in-const branch from 0062f08 to 5a3f418 Compare November 20, 2021 14:32
@xFrednet xFrednet force-pushed the 8004-suboptimal-flops-in-const branch from 5a3f418 to 1c8085d Compare November 20, 2021 14:41
@giraffate
Copy link
Contributor

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Nov 21, 2021

📌 Commit 1c8085d has been approved by giraffate

@bors
Copy link
Contributor

bors commented Nov 22, 2021

⌛ Testing commit 1c8085d with merge 5fbfdfa...

@bors
Copy link
Contributor

bors commented Nov 22, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: giraffate
Pushing 5fbfdfa to master...

@bors bors merged commit 5fbfdfa into rust-lang:master Nov 22, 2021
@xFrednet xFrednet deleted the 8004-suboptimal-flops-in-const branch November 22, 2021 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

suboptimal_flops in const functions
4 participants