<div dir="ltr"><div dir="ltr">On Wed, Jun 10, 2020 at 6:19 PM Tom Stellard <<a href="mailto:tstellar@llvm.org">tstellar@llvm.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 06/10/2020 09:17 AM, Sam McCall wrote:<br>
> Hi Tom,<br>
> <br>
> 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.<br>
> <br>
> 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".<br>
> (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)<br>
> <br>
> 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.<br>
> The commits are this range <<a href="https://github.com/llvm/llvm-project/compare/a634a80615b1e012f1a61aa0cd1e2e67ef77d0bd...b6efa2365812f31667485c8948d49621ebf952f2" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/compare/a634a80615b1e012f1a61aa0cd1e2e67ef77d0bd...b6efa2365812f31667485c8948d49621ebf952f2</a>>. Of these all affect clangd only, except a trivial clang crash fix by Richard Smith <<a href="https://github.com/llvm/llvm-project/commit/d942a81c71a3f970857e92e98cc6503b61b589b0" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/d942a81c71a3f970857e92e98cc6503b61b589b0</a>> and a small tweak to a clang-tidy check <<a href="https://github.com/llvm/llvm-project/commit/d942a81c71a3f970857e92e98cc6503b61b589b0" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/d942a81c71a3f970857e92e98cc6503b61b589b0</a>>. (There are a couple of changes to clang/lib/Syntax that are only used in clangd).<br>
> <br>
> Let me know what you'd prefer (or just go ahead and revert if that's easiest).<br>
> Sorry about the churn.<br>
> <br>
<br>
No problem, thanks for letting me know.  Can you file a bug<br>
with all the commits you cherry-picked and I will take a look.<br>
I think we can leave the commits in for now and revert only if<br>
they don't pass review.<br></blockquote><div>Thank you! I filed <a href="https://bugs.llvm.org/show_bug.cgi?id=46270">https://bugs.llvm.org/show_bug.cgi?id=46270</a> and attempted to summarize. Please let me know what else is needed!</div><div><br></div><div>Haojian and Kadir (CCed) are familiar with the patches and happy to have them merged.</div><div>The clang-proper patches are the ones where we don't have owner-level expertise: I'll ask Richard to sign off on cbc9b92df4582617314b08d1ecef41d355733874 (which is the one he didn't author himself).</div><div><br></div><div>Thanks, Sam</div></div></div>