[PATCH] D146377: [clangd] Remove reundant use of getSpellingLoc()

2023-03-20 Thread Nathan Ridge via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG57bfe25574a0: [clangd] Remove reundant use of getSpellingLoc() (authored by nridge). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146377/new/

[PATCH] D146377: [clangd] Remove reundant use of getSpellingLoc()

2023-03-20 Thread Haojian Wu via Phabricator via cfe-commits
hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146377/new/ https://reviews.llvm.org/D146377

[PATCH] D146377: [clangd] Remove reundant use of getSpellingLoc()

2023-03-19 Thread Nathan Ridge via Phabricator via cfe-commits
nridge added a comment. No functional changes intended, this redundancy is just something I noticed while I was learning about these functions during my review of D144074 and looking at other usages of them in clnagd. Repository: rG LLVM Github Monorepo

[PATCH] D146377: [clangd] Remove reundant use of getSpellingLoc()

2023-03-19 Thread Nathan Ridge via Phabricator via cfe-commits
nridge created this revision. nridge added reviewers: hokein, kadircet. Herald added a subscriber: arphaman. Herald added a project: All. nridge requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra.