Skip to content

Conversation

@xFrednet
Copy link
Contributor

@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