hazohelet wrote:

When the `Loc` is higher than the lowest begin source location, `DisplayLineNo` 
still points to the lowest begin location line because we have
```c++
  DisplayLineNo =
        std::min(DisplayLineNo, SM.getPresumedLineNumber(I.getBegin()));
```
in L1169, correct?
This does not break the test added in the commit at least locally.

I don't think we need to ignore `Loc.getPresumedLoc().getLine()` when `Ranges` 
is not empty.

https://github.com/llvm/llvm-project/pull/65238
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to