[PATCH] D24164: Remove debug info when hoisting instruction from then/else branch.

[Paul Robinson via llvm-commits]

probinson added a subscriber: probinson.
probinson added a comment.

In https://reviews.llvm.org/D24164#533199, @andreadb wrote:

> > About https://reviews.llvm.org/D24180, we just use -mllvm -use-unknown-locations=true for similar purposes.
>
>
> Interesting, I will look at it.
>  At the moment I am using https://reviews.llvm.org/D24180 and it seems to be doing a good job with sample pgo. In my experiments, it fixes a few nasty cases where instructions at the top of a basic block are implicitly attributed to the source line for the physically preceding instruction simply because they don't have a debug loc. Anyway, I will explain all the details in the email that I will send to you :-).


Using `-mllvm -use-unknown-locations=true` is a superset of https://reviews.llvm.org/D24180.  The latter tries to be smarter about where the line-0 attributions are really needed, in order to minimize the size penalty in the debug line table.  Also https://reviews.llvm.org/D24180 works for Codeview as well as DWARF.


https://reviews.llvm.org/D24164