[llvm] r223193 - Emit the entry block first and the exit block second, then all the blocks in between afterwards. This is what gcc always does, and some out of tree tools depend on that.

[Justin Bogner]

Justin Bogner <mail at justinbogner.com> writes:
> Justin Bogner <mail at justinbogner.com> writes:
>> Nick Lewycky <nicholas at mxc.ca> writes:
>> Yep, this behaviour is definitely incompatible with the 402* format. We
>> need to put this behind a flag and emit a newer version (408*, I think -
>> this behaviour doesn't seem to happen with 4.4 or 4.7) when we do this
>> so that llvm-cov doesn't try to read it as the old format. We'll also
>> want to leave the default on Darwin to emit the 402* format.
>>
>> Can you do this? I'm not sure what the appropriate name for the flag is
>> and can't really validate that what we're outputting is correct for a
>> 408* version.
>
> Ping. We should really fix this soon, since we're generating gcov output
> that our own llvm-cov can't even read. See also pr22436.

Nick and I spoke offline and I've added a flag for this in
r232438/r232439, returning the default to the 4.2 compatible behaviour.