Skip to content

Commit 9edce2b

Browse files
committed
Unset lto for fuzz, as it conflicts with codegen-units = 1 anyways
Previously, we set `codegen-units = 1` *and* `lto = true` in `Cargo.toml`, but the docs for `lto` say: ``` No LTO is performed if codegen units is 1 or opt-level is 0. ``` https://doc.rust-lang.org/cargo/reference/profiles.html#lto Hence, we here remove `lto = true` since it's not used anyways and confusing.
1 parent d67bd0f commit 9edce2b

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

fuzz/Cargo.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ cc = "1.0"
3737
members = ["."]
3838

3939
[profile.release]
40-
lto = true
4140
codegen-units = 1
4241
debug-assertions = true
4342
overflow-checks = true

fuzz/ci-fuzz.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ popd
2323
cargo install --color always --force honggfuzz --no-default-features
2424
# Because we're fuzzing relatively few iterations, the maximum possible
2525
# compiler optimizations aren't necessary, so switch to defaults.
26-
sed -i 's/lto = true//' Cargo.toml
2726
sed -i 's/codegen-units = 1//' Cargo.toml
2827

2928
export HFUZZ_BUILD_ARGS="--features honggfuzz_fuzz"

0 commit comments

Comments
 (0)