dyung wrote: > > > > Hmm, comments in #212856 were overlooked. Maybe add an NFC patch before > > > > backporting? > > > > > > > > > My bad: #218142 I don't know if one could merge both at the same time, or > > > if that have to be two cherry-picks. > > > > > > Should we wait to land this cherry-pick until #218142 is also ready to be > > cherry-picked? I assume the latter would be dependent on this one going in > > first? > > We can manually update this patch with the suggestions which are the same as > #218142.
That's fine with me, shall I go ahead and do that? (I suspect only the release managers may be able to update these branches in the llvmbot repo) https://github.com/llvm/llvm-project/pull/216554 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
