Re: [PATCH] Cleanup back_threader::find_path_to_names.

2021-11-05 Thread Jeff Law via Gcc-patches
On 11/5/2021 2:44 PM, Aldy Hernandez wrote: On Fri, Nov 5, 2021 at 9:06 PM Jeff Law wrote: On 11/5/2021 9:09 AM, Aldy Hernandez wrote: The main path discovery function was due for a cleanup. First, there's a nagging goto and second, my bitmap use was sloppy. Hopefully this makes the

Re: [PATCH] Cleanup back_threader::find_path_to_names.

2021-11-05 Thread Aldy Hernandez via Gcc-patches
deed be killed. This was left over from when resolve_phi() didn't resolve all incoming paths, but that's no longer the case. Once we exit find_path_to_names, all paths have been explored and there's nothing to do. OK pending tests? Aldy > > So I think that raises the question, for the recursive cal

Re: [PATCH] Cleanup back_threader::find_path_to_names.

2021-11-05 Thread Jeff Law via Gcc-patches
On 11/5/2021 9:09 AM, Aldy Hernandez wrote: The main path discovery function was due for a cleanup. First, there's a nagging goto and second, my bitmap use was sloppy. Hopefully this makes the code easier for others to read. Regstrapped on x86-64 Linux. I also made sure there were no

[PATCH] Cleanup back_threader::find_path_to_names.

2021-11-05 Thread Aldy Hernandez via Gcc-patches
The main path discovery function was due for a cleanup. First, there's a nagging goto and second, my bitmap use was sloppy. Hopefully this makes the code easier for others to read. Regstrapped on x86-64 Linux. I also made sure there were no difference in the number of threads with this patch.