<div dir="ltr">Hi Tom,<div><br></div><div>We got some user requests for (clangd) cherrypicks in the next point release, which nudged us in digging up a list of other bugfixes (mostly crashers) to backport.</div><div><br></div><div>And this is where I messed up: we reviewed the list and then pushed them to the 10.x branch, forgetting about "Patches applied to the release branch may only be applied ... with approval from the release manager".</div><div>(The excuse: I've mostly been active with major releases, and we had an understanding with Hans that any clangd-only changes were fine up to RC2 or so. Point releases are different of course and this is your call, I'd just forgotten the actual policy)</div><div><br></div><div>Anyway, I'm happy to revert any/all of these changes if you don't think they belong on the branch (or pending a decision). They're all bugfixes and suitable in principle for a point release, but up to you whether they belong in this one.</div><div>The commits are <a href="https://github.com/llvm/llvm-project/compare/a634a80615b1e012f1a61aa0cd1e2e67ef77d0bd...b6efa2365812f31667485c8948d49621ebf952f2">this range</a>. Of these all affect clangd only, except a <a href="https://github.com/llvm/llvm-project/commit/d942a81c71a3f970857e92e98cc6503b61b589b0">trivial clang crash fix by Richard Smith</a> and a small tweak to a <a href="https://github.com/llvm/llvm-project/commit/d942a81c71a3f970857e92e98cc6503b61b589b0">clang-tidy check</a>. (There are a couple of changes to clang/lib/Syntax that are only used in clangd).</div><div><br></div><div>Let me know what you'd prefer (or just go ahead and revert if that's easiest).</div><div>Sorry about the churn.</div><div><br></div><div>Sam</div></div>