Re: [PATCH] PHIOPT: Improve replace_phi_edge_with_variable's dce_ssa_names slightly

2023-05-04 Thread Richard Biener via Gcc-patches
On Thu, May 4, 2023 at 1:11 AM Andrew Pinski via Gcc-patches wrote: > > When I added the dce_ssa_names argument, I didn't realize bitmap was a > pointer so I used the default argument value as auto_bitmap(). But > instead we could just use nullptr and check if it was a nullptr > before calling sim

[PATCH] PHIOPT: Improve replace_phi_edge_with_variable's dce_ssa_names slightly

2023-05-03 Thread Andrew Pinski via Gcc-patches
When I added the dce_ssa_names argument, I didn't realize bitmap was a pointer so I used the default argument value as auto_bitmap(). But instead we could just use nullptr and check if it was a nullptr before calling simple_dce_from_worklist. OK? Bootstrapped and tested on x86_64-linux-gnu with no