<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi,<div><br></div><div>in <a href="https://reviews.llvm.org/D52143">https://reviews.llvm.org/D52143</a> there's some uncertainty if LLVM code prefers</div><div><br></div><div>  if (!Function())</div><div>    // Call to function failed, deal with it</div><div><br></div><div>or</div><div><br></div><div><div>  if (Function())</div><div>    // Call to function failed, deal with it</div></div><div><br></div><div>(Note that this is about functions returning bool, not int.)<br></div><div><br></div><div>Folks on that review feel that returning true on success is probably what we want, but it's not documented anywhere and we do have both forms in the codebase.</div><div><br>True on success seems more common:</div><div><a href="http://llvm-cs.pcc.me.uk/?q=true+on+success">http://llvm-cs.pcc.me.uk/?q=true+on+success</a><br></div><div><a href="http://llvm-cs.pcc.me.uk/?q=true+on+error">http://llvm-cs.pcc.me.uk/?q=true+on+error</a><br></div><div><br></div><div>Does anyone have a pointer to previous on-list discussion on this? If not, this thread could be the place where we sort this out once and for all :-)</div><div><br></div><div>Apologies for the bike-sheddy topic.</div><div><br></div><div>Nico</div></div></div></div></div></div>