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

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 8 10:42:31 PDT 2016


echristo added a comment.

In https://reviews.llvm.org/D24164#537288, @probinson wrote:

> I'm pretty sure DWARF inherently can't attribute the same instruction to two source locations, so I think it is preferable to say "don't know" all the time than to give the wrong answer half the time.


Right, but I'd prefer an answer that's slightly inaccurate to one that attributes the location to a source line that just doesn't even resemble it :)

> I'd be curious how the sanitizers actually handle the line-0 case.


Ditto.


https://reviews.llvm.org/D24164





More information about the llvm-commits mailing list