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