aeubanks added a comment. In D83519#2144463 <https://reviews.llvm.org/D83519#2144463>, @aeubanks wrote:
> In D83519#2144403 <https://reviews.llvm.org/D83519#2144403>, @ychen wrote: > > > High-level request: how about split this patch into two, the first for the > > `require` pass part; the second for the PassInstrument callback. Then we > > could discuss the choices of first patch and D82344 > > <https://reviews.llvm.org/D82344>. > > > Good idea, will split the patch and take a closer look at your patch. I split the required passes part into https://reviews.llvm.org/D83575. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83519/new/ https://reviews.llvm.org/D83519 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
