<div dir="ltr">I'm not sure how easy it is to get clang-format into Phabricator since it is mostly developed by (phab) upstream. I'll file feature request regarding trailing whitespaces and clang-format to upstream to see what upstream forks think. <div><br></div><div>But a bot checking revision sounds great. It might also be useful to have the bot run clang-tidy and post findings/FixHints as comments on the patch. </div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 16, 2016 at 12:05 PM Johannes Doerfert via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 11/14, Mehdi Amini via llvm-dev wrote:<br class="gmail_msg">
> Ideally I’d even really like to have a both checking for revision on<br class="gmail_msg">
> phab, clang-formatting them, and post a comment when there is a<br class="gmail_msg">
> mismatch :)<br class="gmail_msg">
I'd like that!<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
LLVM Developers mailing list<br class="gmail_msg">
<a href="mailto:llvm-dev@lists.llvm.org" class="gmail_msg" target="_blank">llvm-dev@lists.llvm.org</a><br class="gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br class="gmail_msg">
</blockquote></div></div>