ro added a comment. In D90524#2393366 <https://reviews.llvm.org/D90524#2393366>, @glaubitz wrote:
> In D90524#2393320 <https://reviews.llvm.org/D90524#2393320>, @ro wrote: > >> In D90524#2393312 <https://reviews.llvm.org/D90524#2393312>, @glaubitz wrote: >> >>> Ping. It would be nice to get this finally merged so that the testsuite >>> noise finally goes down on the sparc64 Linux worker. >> >> Please be a little more patient: one ping a week is considered appropriate, >> but after only two days is a bit over the top. > > The problem is that LLVM is a very fast moving target and when waiting long > for changes to be merged, one constantly runs > into the risk of having to rebase patches. True, but it's the same for everyone. Constantly pinging at short intervals won't get you faster reviews, but just creates noise and annoys possible reviewers. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90524/new/ https://reviews.llvm.org/D90524 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
