<div><div dir="auto">Well I agree, i also think we should have clear guidelines when possible.  But “when possible” is itself subjective.  We can’t cover every possible scenario because it would be impossible to remember all the rules, so instead we use general guidelines.  So we have to have a balance between rules that are useful while not specifying every detail of what the code should look like, and then in code review there can be a second line of defense for reviewers to point out issues.</div><br><br><div dir="auto">Fwiw, I feel like this patch is a response to comments pointed out by reviewers in </div><div><a href="https://reviews.llvm.org/D54405">https://reviews.llvm.org/D54405</a>, and I think when multiple people have concerns about something, the correct response is to simply address the concerns.  I’m not opposed to modifying the rules around auto, but I don’t think it should be used as a way to avoid addressing reviewer feedback.</div></div><div dir="auto"><br></div><div dir="auto"><br></div><div><div class="gmail_quote"><div dir="ltr">On Mon, Nov 26, 2018 at 8:35 PM Mehdi AMINI via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">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">mehdi_amini added a comment.<br>
<br>
In D54877#1309017 <<a href="https://reviews.llvm.org/D54877#1309017" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54877#1309017</a>>, @zturner wrote:<br>
<br>
> In D54877#1308855 <<a href="https://reviews.llvm.org/D54877#1308855" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54877#1308855</a>>, @zturner wrote:<br>
><br>
> ><br>
><br>
><br>
> at the end of the day, comments that arise in code review should take precedence over what the style guide does or doesn't say when it concerns readability, since it's an actual account of someone stating that they do or don't find something readable, and that's what we're trying to optimize for.<br>
<br>
<br>
FWIW as another data point: I have a totally opposed view on the subject: I rather have clear guidelines *when possible*. Otherwise it is the realm of inconsistency and instability: one reviewers makes you change towards a prefered idiom while the next reviewer  will prefer another one.<br>
Just because like you wrote before (with a twist): " because there is only 1 of this reviewers and there are many other people".<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D54877/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54877/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D54877" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54877</a><br>
<br>
<br>
<br>
</blockquote></div>
</div>