ychen added a comment. In D83519#2144616 <https://reviews.llvm.org/D83519#2144616>, @aeubanks wrote:
> 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. Could you remove the `Required` part of this patch? 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
