carlosgalvezp added a comment. In D141000#4026600 <https://reviews.llvm.org/D141000#4026600>, @Eugene.Zelenko wrote:
> Looks OK for me, but probably will be good idea to remove check-specific > options in same commit. Thanks for the review! Sure, I can do that, I just thought we should try to keep commits as minimal as possible. I also wonder if we need to follow a formal deprecation process or we can just remove the option from existing checks directly? @njames93 , as Code Owner, could you give your view on the general direction of the patch before I move forward? Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141000/new/ https://reviews.llvm.org/D141000 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
