<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 22, 2016 at 8:22 AM, Robinson, Paul via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
> -----Original Message-----<br>
> From: llvm-commits [mailto:<a href="mailto:llvm-commits-bounces@lists.llvm.org">llvm-commits-bounces@<wbr>lists.llvm.org</a>] On Behalf<br>
</span><span class="">> Of Robinson, Paul via llvm-commits<br>
> Sent: Tuesday, November 22, 2016 6:39 AM<br>
> To: <a href="mailto:reviews%2BD26943%2Bpublic%2B5c1a379143b6f3e8@reviews.llvm.org">reviews+D26943+public+<wbr>5c1a379143b6f3e8@reviews.llvm.<wbr>org</a>; Matt<br>
> Arsenault; <a href="mailto:jlebar@google.com">jlebar@google.com</a>; <a href="mailto:rnk@google.com">rnk@google.com</a>; <a href="mailto:matze@braunis.de">matze@braunis.de</a><br>
> Cc: <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
</span><div><div class="h5">> Subject: RE: D26943: [CodingStandards] Add style guide rule about "if"<br>
> statements and loops.<br>
><br>
><br>
><br>
> > -----Original Message-----<br>
> > From: llvm-commits [mailto:<a href="mailto:llvm-commits-bounces@lists.llvm.org">llvm-commits-bounces@<wbr>lists.llvm.org</a>] On<br>
> Behalf<br>
> > Of Matt Arsenault via llvm-commits<br>
> > Sent: Monday, November 21, 2016 4:02 PM<br>
> > To: <a href="mailto:jlebar@google.com">jlebar@google.com</a>; <a href="mailto:rnk@google.com">rnk@google.com</a>; <a href="mailto:matze@braunis.de">matze@braunis.de</a><br>
> > Cc: <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> > Subject: [PATCH] D26943: [CodingStandards] Add style guide rule about<br>
> "if"<br>
> > statements and loops.<br>
> ><br>
> > arsenm added a comment.<br>
> ><br>
> > In <a href="https://reviews.llvm.org/D26943#601970" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26943#601970</a>, @MatzeB wrote:<br>
> ><br>
> > > In <a href="https://reviews.llvm.org/D26943#601961" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26943#601961</a>, @compnerd wrote:<br>
> > ><br>
> > > > Personally, I think I would prefer that we add braces only if one of<br>
> > the bodies of the code path requires it:<br>
> > ><br>
> > ><br>
> > > Just for the sake of discussion (I personally would prefer braces).<br>
> This<br>
> > would result in this in the extreme case:<br>
> > ><br>
> > >   if (foo)<br>
> > >     for (Baz b : bazes)<br>
> > >       if (b)<br>
> > >         ++BCount;<br>
> > >       else<br>
> > >         ++NotBCount;<br>
> > >   else<br>
> > >     foobar();<br>
> > ><br>
> ><br>
> ><br>
> > +1 for braces<br>
><br>
> +1.  In this example both the outermost 'if' and the 'for' have<br>
> a multiline body (even if each individual statement has only 1 line)<br>
> and so I also prefer they have braces.<br>
<br>
</div></div>Although, for really simple cases it's probably fine to omit them:<br>
<br>
  if (auto Foo = bar())<br>
    for (auto Baz : Foo.stuff())<br>
      bazIt(Baz);<br></blockquote><div><br></div><div>These types of constructs are one of my favorite parts of idiomatic LLVM code. I'd hate to lose them.</div><div><br></div><div>My 2cents.</div><div><br></div><div>-- Sean Silva</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
> --paulr<br>
><br>
> ><br>
> ><br>
> > <a href="https://reviews.llvm.org/D26943" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26943</a><br>
> ><br>
> ><br>
> ><br>
> > ______________________________<wbr>_________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
> ______________________________<wbr>_________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>