<div dir="ltr">I believe Justin Bogner went ahead and changed all the lines in SimplifyCFG to || in r340153.<div><br clear="all"><div><div dir="ltr" class="m_8451309559317909360gmail_signature" data-smartmail="gmail_signature">~Craig</div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 22, 2018 at 9:30 PM Chris Lattner <<a href="mailto:clattner@nondot.org" target="_blank">clattner@nondot.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space">On Aug 19, 2018, at 10:33 PM, Craig Topper via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<div><blockquote type="cite"><div><div dir="ltr"><div>The language requires both sides of the | need to be evaluated. The compiler can't remove the function call if it has side effects.</div></div></div></blockquote><div><br></div><div>Correct.  If you are somehow really bothered by this, we could also use:</div><div><br></div><div><span style="font-family:Verdana,serif;font-size:13.333333015441895px">bool bar() {</span><br style="font-family:Verdana,serif;font-size:13.333333015441895px"><span style="font-family:Verdana,serif;font-size:13.333333015441895px">  return foo(), true;</span></div><div><span style="font-family:Verdana,serif;font-size:13.333333015441895px">}</span></div><div><span style="font-family:Verdana,serif;font-size:13.333333015441895px"><br></span></div><div><font face="Verdana, serif" size="2">-Chris</font></div></div></div></blockquote></div>