Nathan Reynolds

I looked through Jacoco code. I think the code change for adding the `if (jacocoData[3] == 0)` will happen in `ProbeInserter.insertProbe()`.

If you create an agent jar with this code change, I am willing to check it out and see if it performs better. I am hopeful that the performance impact...

My first Jacoco optimization was to exclude as much as possible. This helped since I removed all of the test methods. Thanks for the pointer though. Thanks for the pointer...

> For tests that do not warm code, the performance is going to degrade by 6 cycles per probe versus the current solution. 3 cycles to load the value from...

@marchof I am testing on Java 10

@Godin Thanks for the zip file. I will try it out. It will take a few hours.

@Godin Thanks for the pointers. This conversation has been enlightening.

Here are the awesome results of running with and without Jacoco. The CPU is the average CPU usage for the entire test time. 100% means 1 core fully utilized. The...

The code uses a boolean array to store the coverage. Have you considered using static byte fields to store the coverage? (Do not use boolean fields since these are encoded...

Thank you for the insights. It seems calling a static method and conditionally setting the boolean array element to true is the best solution (i.e. simulated_proposed_branch_in_method). Thank you for the...