Signed-off-by: Brian Anderson <andersrb@gmail.com>
As in the Makefile Signed-off-by: Brian Anderson <andersrb@gmail.com>
Makes it clear exactly what our settings are. These are the values documented for the current stable compiler. Signed-off-by: Brian Anderson <andersrb@gmail.com>
Benchmarking without any LTO seems pretty bogus. This will make compile time for `cargo bench` worse. Signed-off-by: Brian Anderson <andersrb@gmail.com>
This should be the single biggest compile time savings. It is not disabled for the release build yet but will be once the Jenkins release builders are modified to use a different profile. For now README.md mentions how to turn on debuginfo, but I will move that to the developer's guide once _that_ exists. Signed-off-by: Brian Anderson <andersrb@gmail.com>
Per discussion with jay we'll leave incremental off of the release profile. Signed-off-by: Brian Anderson <andersrb@gmail.com>
Signed-off-by: Brian Anderson <andersrb@gmail.com>
Open
Merged
nrc previously approved these changes Jun 6, 2019
Signed-off-by: Brian Anderson <andersrb@gmail.com>
Signed-off-by: Brian Anderson <andersrb@gmail.com>
Open
breezewish pushed a commit to breezewish/tikv that referenced this pull request
Jun 24, 2019
…ikv#4689) * Makefile: fix x-bench and x-test Signed-off-by: Brian Anderson <andersrb@gmail.com> * run-cargo.sh: use --no-default-features As in the Makefile Signed-off-by: Brian Anderson <andersrb@gmail.com> * Cargo.toml: add the full set of profile options Makes it clear exactly what our settings are. These are the values documented for the current stable compiler. Signed-off-by: Brian Anderson <andersrb@gmail.com> * Cargo.toml: set lto=thin for profile.bench Benchmarking without any LTO seems pretty bogus. This will make compile time for `cargo bench` worse. Signed-off-by: Brian Anderson <andersrb@gmail.com> * Cargo.toml: turn off debuginfo for all profiles except release This should be the single biggest compile time savings. It is not disabled for the release build yet but will be once the Jenkins release builders are modified to use a different profile. For now README.md mentions how to turn on debuginfo, but I will move that to the developer's guide once _that_ exists. Signed-off-by: Brian Anderson <andersrb@gmail.com> * etc: turn off incremental for dev-opt profile Per discussion with jay we'll leave incremental off of the release profile. Signed-off-by: Brian Anderson <andersrb@gmail.com> * Cargo.toml: set debug=1 for test profile Signed-off-by: Brian Anderson <andersrb@gmail.com> * README: fix -Cdebuginfo usage Signed-off-by: Brian Anderson <andersrb@gmail.com> * README: spelling Signed-off-by: Brian Anderson <andersrb@gmail.com>
Closed
sticnarf pushed a commit to sticnarf/tikv that referenced this pull request
Oct 27, 2019
…ikv#4689) * Makefile: fix x-bench and x-test Signed-off-by: Brian Anderson <andersrb@gmail.com> * run-cargo.sh: use --no-default-features As in the Makefile Signed-off-by: Brian Anderson <andersrb@gmail.com> * Cargo.toml: add the full set of profile options Makes it clear exactly what our settings are. These are the values documented for the current stable compiler. Signed-off-by: Brian Anderson <andersrb@gmail.com> * Cargo.toml: set lto=thin for profile.bench Benchmarking without any LTO seems pretty bogus. This will make compile time for `cargo bench` worse. Signed-off-by: Brian Anderson <andersrb@gmail.com> * Cargo.toml: turn off debuginfo for all profiles except release This should be the single biggest compile time savings. It is not disabled for the release build yet but will be once the Jenkins release builders are modified to use a different profile. For now README.md mentions how to turn on debuginfo, but I will move that to the developer's guide once _that_ exists. Signed-off-by: Brian Anderson <andersrb@gmail.com> * etc: turn off incremental for dev-opt profile Per discussion with jay we'll leave incremental off of the release profile. Signed-off-by: Brian Anderson <andersrb@gmail.com> * Cargo.toml: set debug=1 for test profile Signed-off-by: Brian Anderson <andersrb@gmail.com> * README: fix -Cdebuginfo usage Signed-off-by: Brian Anderson <andersrb@gmail.com> * README: spelling Signed-off-by: Brian Anderson <andersrb@gmail.com>