bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.



> `ninja check-clang` passes... is there anything else I should be testing?

I'm not sure, just double checking :)

The general approach LGTM though.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58890/new/

https://reviews.llvm.org/D58890



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D58890: Mo... Duncan P. N. Exon Smith via Phabricator via cfe-commits
    • [PATCH] D5889... Bruno Cardoso Lopes via Phabricator via cfe-commits
    • [PATCH] D5889... Duncan P. N. Exon Smith via Phabricator via cfe-commits
    • [PATCH] D5889... Bruno Cardoso Lopes via Phabricator via cfe-commits
    • [PATCH] D5889... Duncan P. N. Exon Smith via Phabricator via cfe-commits

Reply via email to