On 7/16/24 4:34 PM, Andrew Pinski wrote:
Like r15-1610-gb6215065a5b143 (which adds one for late_combine),
adding one for ext_dce is useful to debug some issues with this pass.
Bootstrapped and tested on x86_64-linux-gnu with no regressions.
gcc/ChangeLog:
* dbgcnt.def (ext_dce): New debug counter.
* ext-dce.cc (ext_dce_try_optimize_insn): Reject the insn
if the debug counter says so.
This is, of course, fine. I've got a potential patch for the -Os issue
that seems to be working. Debug counters are definitely useful.
I think it's probably best to wait for me to wrap up the fix for that
synthetic case, then go back and re-test harfbuzz. I just had a long
unexpected interruption today, so I didn't get the time to walk over
that fix again to make sure I was happy with it.
jeff
