arphaman accepted this revision. arphaman added a subscriber: NoQ. arphaman added a comment. This revision is now accepted and ready to land. Herald added a subscriber: MaskRay.
LGTM, with one request. Can you please take the static analyzer changes (RetainSummaryManager.h/cpp) and make them into a separate patch? Please add @NoQ as a reviewer for that change. Then feel free to land the remaining driver kit changes in this patch as one commit. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121911/new/ https://reviews.llvm.org/D121911 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits