[PATCH] D110081: [llvm-profgen] Filter out invalid debug line

Fri Oct 1 00:14:10 PDT 2021

wenlei accepted this revision.
wenlei added a comment.

In D110081#3028373 <https://reviews.llvm.org/D110081#3028373>, @wlei wrote:

> In D110081#3010969 <https://reviews.llvm.org/D110081#3010969>, @wenlei wrote:
>
>> I think I've seen negative line offsets due to use of macro. If macro is used the line info may point to location where the macro is defined, then it can lead to negative line offset if we use function start line as the base.
>>
>> In such cases, negative offset should still work for correlating profile back to IR.. What is the corresponding line offset you saw on relevant IR? Do they match the offset in the profile?
>
> As we discussed offline, we might have the invalid number in non-macro case and our internal tools also haven't covered this which can be a separate improvements later.

Okay, as long as this matches the behavior of original tool, this looks good. I'm also curious how google's tool deal with such negative offset though..

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110081/new/

https://reviews.llvm.org/D110081