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

Reply via email to