ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land.
@kadircet says we're using this pattern all over clangd. This somehow slipped my view before. LGTM to be consistent with the rest of clangd (and functions in `FindTarget.h`). I think we should switch to passing non-const pointers everywhere, but that's a discussion for another day... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68027/new/ https://reviews.llvm.org/D68027 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
