On 1/22/21 2:07 PM, Jan Hubicka wrote:
This is OK. To save future debugging, perhaps I would keep the code printing the tp first run value to dump file and do fprintf (dump_file, "Read tp_first_run: %d; ignored because profile reproducibility is multithreaded\n", node->tp_first_run); In few years we may forget about this logic and wonder why it does not work...
Sure, that's a reasonable dump message. I'm going to push the commit. Martin