Skip to content

Commit 59bac6a

Browse files
committed
Auto merge of rust-lang#13408 - y21:quine-dev-o3, r=dswij
Build quine-mc_cluskey with `opt-level=3` in dev builds While doing some profiling I noticed that debug clippy running on the `clippy_lints` crate spends 35s out of 160s in one specific code path of `nonminimal_bool`, which seemed a bit excessive. I've found that just enabling optimizations for quine-mc_cluskey (used by nonminimal_bool) cuts down the part that took 35s to 3s While this doesn't really change anything for users, this helps dogfood a bit as it cuts off about half a minute of runtime (in some of my tests, at least). Something similar was attempted in rust-lang#10576, however that involved compiling everything in release mode including clippy itself, whereas this only affects a single dependency that's compiled in parallel with something that takes longer so this should hopefully not have a negative impact in any case (and changing clippy doesn't require recompiling that dependency) changelog: none
2 parents 43e3384 + 431f7d6 commit 59bac6a

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

Cargo.toml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,3 +67,10 @@ harness = false
6767
[[test]]
6868
name = "dogfood"
6969
harness = false
70+
71+
# quine-mc_cluskey makes up a significant part of the runtime in dogfood
72+
# due to the number of conditions in the clippy_lints crate
73+
# and enabling optimizations for that specific dependency helps a bit
74+
# without increasing total build times.
75+
[profile.dev.package.quine-mc_cluskey]
76+
opt-level = 3

0 commit comments

Comments
 (0)