<div><div dir="auto">Thank you for handling that</div></div><div dir="auto"><br></div><div dir="auto">I will look into your suggestions </div><div dir="auto"><br></div><div dir="auto">MyDeveloperDay</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 12 Oct 2019 at 23:55, Nico Weber via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">thakis added a comment.<br>
<br>
This fails on macOS:<br>
<br>
  : 'RUN: at line 2';   grep -E "*code should be clang-formatted*" /Users/thakis/src/llvm-project/out/gn/obj/clang/test/Format/Output/dry-run.cpp.tmp.stderr<br>
  --<br>
  Exit Code: 2<br>
<br>
  Command Output (stderr):<br>
  --<br>
  grep: repetition-operator operand invalid<br>
<br>
grep -E means extended regex, where you want `.*` to match anything. But this probably should use FileCheck, or maybe even -verify since you're adding a dep on Frontend anyways.<br>
<br>
I've reverted this for now to green up the bots.<br>
<br>
For the reland, did y'all consider the impact of this on clang-format build time (it now depends on Frontend and hence on Driver and Sema, and Sema is slow to build) and binary size (it's now basically impossible to ever get the diagnostics tables in clang-format gc'd by the linker)?<br>
<br>
It might make sense to instead use LLVM's diag stuff (instead of clang's) since all the actual diags will be disjoint.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D68554/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D68554/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D68554" rel="noreferrer" target="_blank">https://reviews.llvm.org/D68554</a><br>
<br>
<br>
<br>
</blockquote></div></div>