<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 11, 2018 at 3:43 PM Ilya Biryukov 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">ilya-biryukov added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D51747#1229066" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51747#1229066</a>, @sammccall wrote:<br>
<br>
> A few thoughts here:<br>
><br>
> - does CDB describe user or project preferences? unclear.<br>
<br>
<br>
Agree, it's a mix, defaults are from the project but users can add extra flags.<br>
<br>
> - "show this warning for code I build" is a higher bar than "show this warning for code I edit". So CDB probably enables too few warnings.<br>
> - Some warnings play well with -Werror (like uninit warnings), some don't (like deprecated). -Werror projects often disable interesting warnings.<br>
<br>
Agreed, editors are different from build.<br>
<br>
> I'm not sure we should strictly follow the CDB, but the bar to override it should probably be high.<br>
<br>
WDYT in the long term about a more general mechanism (to allow users override compiler or warning flags at the clangd level?<br>
So that even if clangd is opinionated about the default warnings it enables, users have an option to override according to their preferences.<br></blockquote><div>Yeah, I can see making "extra clang flags" a clangd flag (at some point we really need .clangd config file or something...)</div><div><br></div><div>The scary thing about the extra flags is how they interact with driver mode (clang-cl vs clang), but maybe that's the user's problem.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
In <a href="https://reviews.llvm.org/D51747#1230420" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51747#1230420</a>, @kadircet wrote:<br>
<br>
> if user wants to see all diagnostics as a list suddenly they will get deprecations in that list as well :(.<br>
<br>
<br>
Yeah, some level of noise is probably inevitable.<br>
<br>
<br>
Repository:<br>
  rCTE Clang Tools Extra<br>
<br>
<a href="https://reviews.llvm.org/D51747" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51747</a><br>
<br>
<br>
<br>
</blockquote></div></div>