melver added a comment. In D103159#2784926 <https://reviews.llvm.org/D103159#2784926>, @aaron.ballman wrote:
> In D103159#2784845 <https://reviews.llvm.org/D103159#2784845>, @melver wrote: > >> Ping. > > FWIW, the usual practice is to ping after no activity on the review for about > a week. > > That said, LGTM! Thanks! And sorry for the early Ping. We had an earlier version of a patch using the no_sanitize("coverage") attribute in Linux -next and it was causing build robots to complain because they somehow want to use a pre-release of Clang 13 (they probably shouldn't, and also probably not a big deal about them suddenly crashing as they should just upgrade). In any case, I thought we should get this fixed sooner than later -- and I just sent this: https://lkml.kernel.org/r/20210527162655.3246381-1-el...@google.com Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103159/new/ https://reviews.llvm.org/D103159 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
