<div dir="ltr">> <span style="font-size:12.8px">Thanks for looking at this. This change needs tests - take a look at the</span><br style="font-size:12.8px"><span style="font-size:12.8px">> gcov tests in tests/tools/llvm-cov to see how these are done.</span><br style="font-size:12.8px"><br>I'd have to add gcda/gcno files generated using recent gcc or clang with custom command-line settings (not sure if clang allows to set the flags I need atm).<br style="font-size:12.8px"><div>I'm not sure how to submit binary files using Phabricator - is this possible without using arc? So far I was using git diff to work with web interface.</div><div><br></div><div>> I'm not a big fan of duplicating this condition here - it'd be better to</div><div>> handle this near where we already read the CfgChecksum so that it's<br>> harder to introduce a bug by causing the condition checks to diverge.<br></div><div><br></div><div>Sure, I can reword this in terms of "HeaderWordsRead". I initially had just one condition but did not like the fact that it updates the min. length as it reads the header; HeaderWordsRead is a better name that makes it cleaner though.</div><div><br></div><div>Arseny</div></div>