[PATCH] D53400: [clangd] Remove the overflow log.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 18 08:04:13 PDT 2018


sammccall added a comment.

After looking at the examples, I'm reassured that we don't really care about tracking precise locations of those references :-)

I'd suggest adding the logging just to where the field is *read* in XRefs.cpp (check if it's max-value).
That would cover a bunch of the cases where an incorrect location is likely to be visible.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53400





More information about the cfe-commits mailing list