lebedev.ri added a comment. In D65300#1647775 <https://reviews.llvm.org/D65300#1647775>, @paulkirth wrote:
> In D65300#1647746 <https://reviews.llvm.org/D65300#1647746>, @lebedev.ri > wrote: > > > Please can you clarify hat's the current layout of these patches? > > Is this patch required, or is it superseded by D66324 > > <https://reviews.llvm.org/D66324> (and thus should be abandoned)? > > I'd like to begin reviewing, but i don't understand where to start. > > > Sorry for the confusion. D66324 <https://reviews.llvm.org/D66324> should > supersede this patch. Okay, i'll be reviewing that one then. > It re-implements everything in the backend, and adds support for IR & Sample > based profiles, so I think it is safe to ignore this patch for now. > Is there something I should do to mark/change the patches? //Please// abandon this patch to make it obvious (Add Action -> Abandon) > Also, thanks for the feedback. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65300/new/ https://reviews.llvm.org/D65300 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
