On Mon, Jun 14, 2021 at 4:04 PM Xinliang David Li <davi...@google.com> wrote:
> > > On Mon, Jun 14, 2021 at 3:59 PM Roman Lebedev via Phabricator < > revi...@reviews.llvm.org> wrote: > >> lebedev.ri added a subscriber: MaskRay. >> lebedev.ri added a comment. >> >> In D104099#2815531 <https://reviews.llvm.org/D104099#2815531>, @wenlei >> wrote: >> >> > In D104099#2814167 <https://reviews.llvm.org/D104099#2814167>, >> @davidxl wrote: >> > >> >> Adding Wei to help measure performance impact on our internal >> workloads. Also add Wenlei to help measure impact with FB's workloads. >> > >> > Measured perf using FB internal workload w/ and w/o this pass, result >> is neutral. >> >> Thank you for checking! >> >> So far, it seems the reaction to this proposal has been overwhelmingly >> positive. >> Does anyone else wish to chime in? Should i land this? @asbirlea @MaskRay >> ? >> > > Wei is doing more measurement @google. Please wait for the response. > > David > Start doing the test. Will report back. Wei. > > >> >> Repository: >> rG LLVM Github Monorepo >> >> CHANGES SINCE LAST ACTION >> https://reviews.llvm.org/D104099/new/ >> >> https://reviews.llvm.org/D104099 >> >>
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
