kiranchandramohan added a comment. In D147218#4645055 <https://reviews.llvm.org/D147218#4645055>, @skatrak wrote:
> Thank you for the review. After I address your last comment my plan is to > land this and the other two accepted REQUIRES patches that depended on it. > > Is there a preferred approach as to how to go about it? I could rebase them > all and wait until the pre-merge builds finish without errors and then land > them in quick succession or I could go one by one to make sure post-merge > builds don't find any issues before landing the next, which will be over a > couple of days most likely. Both are fine. I would recommend the former. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147218/new/ https://reviews.llvm.org/D147218 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
