Re: [COMMITTED 4/4] - Gimple range PHI analyzer and testcases

2023-05-25 Thread Andrew MacLeod via Gcc-patches
On 5/25/23 03:03, Richard Biener wrote: On Wed, May 24, 2023 at 11:21 PM Andrew MacLeod via Gcc-patches There is about a 1.5% slowdown to VRP to invoke and utilize the analyzer in all 3 passes of VRP. overall compile time is 0.06% slower. Bootstraps on x86_64-pc-linux-gnu with no

Re: [COMMITTED 4/4] - Gimple range PHI analyzer and testcases

2023-05-25 Thread Aldy Hernandez via Gcc-patches
Some minor nits. +// There can be only one running at a time. +static phi_analyzer *phi_analysis_object = NULL; Shouldn't this be phi_analyzer_object to be more consistent? Similarly throughout. +// Create a new phi_group with members BM, initialvalue INIT_VAL, modifier +// statement

Re: [COMMITTED 4/4] - Gimple range PHI analyzer and testcases

2023-05-25 Thread Richard Biener via Gcc-patches
On Wed, May 24, 2023 at 11:21 PM Andrew MacLeod via Gcc-patches wrote: > > This patch provide the framework for a gimple-range phi analyzer. > > Currently, the primary purpose is to give better initial values for > members of a "phi group" > > a PHI group is defined as a a group of PHI nodes

[COMMITTED 4/4] - Gimple range PHI analyzer and testcases

2023-05-24 Thread Andrew MacLeod via Gcc-patches
This patch provide the framework for a gimple-range phi analyzer. Currently, the  primary purpose is to give better initial values for members of a "phi group" a PHI group is defined as a a group of PHI nodes whose arguments are all either members of the same PHI group, or one of 2 other