<div dir="ltr">I was more thinking about excluding some/not all files in certain directories rather than excluding the whole directory/tree (ala .gitignore pattern), but you are correct it could be disabled in this way.<div><br></div><div>MyDeveloperDay.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 1, 2020 at 8:31 PM Sam Elliott <<a href="mailto:selliott@lowrisc.org">selliott@lowrisc.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;">You can achieve the same as a `.clang-format-ignore` for vendored directories by putting a `.clang-format` in those directories which turns off formatting (IIRC, this takes two lines). What's more, this works with old versions of clang-format!<div><br></div><div>Sam<br><div><br><blockquote type="cite"><div>On 1 Jul 2020, at 8:22 pm, MyDeveloper Day via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:</div><br><div><div dir="ltr">We recently had a suggestion about introducing a .clang-format-ignore file, it seems there could be value in that for areas of which you don't want to ever format.<div><br></div><div>Anyone undertaking a global clang-format without prior knowledge of the specific area wouldn't necessarily know that.</div><div><br></div><div>MyDeveloperDay</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 1, 2020 at 6:14 PM Michael Kruse <<a href="mailto:llvmdev@meinersbur.de" target="_blank">llvmdev@meinersbur.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Am Mi., 1. Juli 2020 um 05:46 Uhr schrieb MyDeveloper Day via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>>:<br>
> I always knew "polly" was mostly clean too, and actually one of the areas I already test against, (along with lib/Format obviously).<br>
<br>
You have no other choice.<br>
Polly verifies its source formatting as part of check-polly, so every<br>
commit that violates formatting will get an email from the buildbot.It<br>
also triggers when clang-format itself changes. Did you find those<br>
helpful?<br>
<br>
The parts of Polly that are left out are its external dependencies,<br>
i.e. isl and imath. I wonder why these are even included in the<br>
clang-format status, we certainly do not want to modify these source<br>
files.<br>
<br>
For clang, I have to exclude the test folder because clang-format<br>
messes with clang -verify tests that require comments on specific<br>
lines and tests that check correct behavior of clang on whitespace.<br>
<br>
<br>
Michael<br>
</blockquote></div>
_______________________________________________<br>LLVM Developers mailing list<br><a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br></div></blockquote></div><br><div>
<div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div dir="auto" style="color:rgb(0,0,0);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><div>--</div><div>Sam Elliott</div><div>Software Team Lead</div><div>Senior Software Developer - LLVM and OpenTitan</div><div>lowRISC CIC</div></div></div></div></div></div></div></div></div></div></div></div>
</div>
<br></div></div></blockquote></div>