gamesh411 wrote: @steakhal I would like to ask your opinion regarding whether we should actually enable loop-unrolling by default. This is in light of that there is a definite ~4% slowdown all around for 29 new reports and 47 resolved reports out of 3000+ reports all around that is ~1% difference for a ~4% slowdown. The quality of the reports I haven't checked in detail, most of them have a long bugpath.
Here are the reports: | Project | New Reports | Resolved Reports | |---------|-------------|------------------| | memcached | [0 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=memcached_1.6.38_loop_unrolling_experiment2_baseline_8&newcheck=memcached_1.6.38_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=memcached_1.6.38_loop_unrolling_experiment2_baseline_8&newcheck=memcached_1.6.38_loop_unrolling_experiment2_new_8&diff-type=Resolved) | tmux | [1 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=tmux_3.5a_loop_unrolling_experiment2_baseline_8&newcheck=tmux_3.5a_loop_unrolling_experiment2_new_8&diff-type=New) | [1 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=tmux_3.5a_loop_unrolling_experiment2_baseline_8&newcheck=tmux_3.5a_loop_unrolling_experiment2_new_8&diff-type=Resolved) | curl | [1 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=curl_curl-8_12_1_loop_unrolling_experiment2_baseline_8&newcheck=curl_curl-8_12_1_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=curl_curl-8_12_1_loop_unrolling_experiment2_baseline_8&newcheck=curl_curl-8_12_1_loop_unrolling_experiment2_new_8&diff-type=Resolved) | twin | [2 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=twin_v0.9.0_loop_unrolling_experiment2_baseline_8&newcheck=twin_v0.9.0_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=twin_v0.9.0_loop_unrolling_experiment2_baseline_8&newcheck=twin_v0.9.0_loop_unrolling_experiment2_new_8&diff-type=Resolved) | vim | [0 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=vim_v9.1.1232_loop_unrolling_experiment2_baseline_8&newcheck=vim_v9.1.1232_loop_unrolling_experiment2_new_8&diff-type=New) | [36 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=vim_v9.1.1232_loop_unrolling_experiment2_baseline_8&newcheck=vim_v9.1.1232_loop_unrolling_experiment2_new_8&diff-type=Resolved) | openssl | [1 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=openssl_openssl-3.4.1_loop_unrolling_experiment2_baseline_8&newcheck=openssl_openssl-3.4.1_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=openssl_openssl-3.4.1_loop_unrolling_experiment2_baseline_8&newcheck=openssl_openssl-3.4.1_loop_unrolling_experiment2_new_8&diff-type=Resolved) | sqlite | [2 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=sqlite_version-3.49.1_loop_unrolling_experiment2_baseline_8&newcheck=sqlite_version-3.49.1_loop_unrolling_experiment2_new_8&diff-type=New) | [1 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=sqlite_version-3.49.1_loop_unrolling_experiment2_baseline_8&newcheck=sqlite_version-3.49.1_loop_unrolling_experiment2_new_8&diff-type=Resolved) | ffmpeg | [12 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=ffmpeg_n7.1.1_loop_unrolling_experiment2_baseline_8&newcheck=ffmpeg_n7.1.1_loop_unrolling_experiment2_new_8&diff-type=New) | [4 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=ffmpeg_n7.1.1_loop_unrolling_experiment2_baseline_8&newcheck=ffmpeg_n7.1.1_loop_unrolling_experiment2_new_8&diff-type=Resolved) | postgres | [4 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=postgres_REL_17_4_loop_unrolling_experiment2_baseline_8&newcheck=postgres_REL_17_4_loop_unrolling_experiment2_new_8&diff-type=New) | [2 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=postgres_REL_17_4_loop_unrolling_experiment2_baseline_8&newcheck=postgres_REL_17_4_loop_unrolling_experiment2_new_8&diff-type=Resolved) | tinyxml2 | [0 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=tinyxml2_11.0.0_loop_unrolling_experiment2_baseline_8&newcheck=tinyxml2_11.0.0_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=tinyxml2_11.0.0_loop_unrolling_experiment2_baseline_8&newcheck=tinyxml2_11.0.0_loop_unrolling_experiment2_new_8&diff-type=Resolved) | libwebm | [0 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=libwebm_libwebm-1.0.0.31_loop_unrolling_experiment2_baseline_8&newcheck=libwebm_libwebm-1.0.0.31_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=libwebm_libwebm-1.0.0.31_loop_unrolling_experiment2_baseline_8&newcheck=libwebm_libwebm-1.0.0.31_loop_unrolling_experiment2_new_8&diff-type=Resolved) | xerces | [1 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=xerces_v3.3.0_loop_unrolling_experiment2_baseline_8&newcheck=xerces_v3.3.0_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=xerces_v3.3.0_loop_unrolling_experiment2_baseline_8&newcheck=xerces_v3.3.0_loop_unrolling_experiment2_new_8&diff-type=Resolved) | bitcoin | [0 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=bitcoin_v28.1_loop_unrolling_experiment2_baseline_8&newcheck=bitcoin_v28.1_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=bitcoin_v28.1_loop_unrolling_experiment2_baseline_8&newcheck=bitcoin_v28.1_loop_unrolling_experiment2_new_8&diff-type=Resolved) | protobuf | [0 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=protobuf_v30.1_loop_unrolling_experiment2_baseline_8&newcheck=protobuf_v30.1_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=protobuf_v30.1_loop_unrolling_experiment2_baseline_8&newcheck=protobuf_v30.1_loop_unrolling_experiment2_new_8&diff-type=Resolved) | qtbase | [6 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=qtbase_v6.8.2_loop_unrolling_experiment2_baseline_8&newcheck=qtbase_v6.8.2_loop_unrolling_experiment2_new_8&diff-type=New) | [3 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=qtbase_v6.8.2_loop_unrolling_experiment2_baseline_8&newcheck=qtbase_v6.8.2_loop_unrolling_experiment2_new_8&diff-type=Resolved) | contour | [0 new reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=contour_v0.6.1.7494_loop_unrolling_experiment2_baseline_8&newcheck=contour_v0.6.1.7494_loop_unrolling_experiment2_new_8&diff-type=New) | [0 resolved reports](https://codechecker-demo.eastus.cloudapp.azure.com/Default/reports?run=contour_v0.6.1.7494_loop_unrolling_experiment2_baseline_8&newcheck=contour_v0.6.1.7494_loop_unrolling_experiment2_new_8&diff-type=Resolved) https://github.com/llvm/llvm-project/pull/178911 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
