<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, 11 Jul 2016 at 14:02 Obiesie ike-nwosu via Python-Dev <<a href="mailto:python-dev@python.org">python-dev@python.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"><font face="Monaco" style="font-size:11px">Hi,</font><div><font face="Monaco" style="font-size:11px"><br></font></div><div><font face="Monaco" style="font-size:11px">I am looking into how the python compiler generates basic blocks during the CFG generation process and my expectations from CFG theory seems to be at odds with how the python compiler actually generates its CFG. Take the following code snippet for example:</font></div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><div><font face="Courier">def median(pool):</font></div></div><div><div><font face="Courier">    copy = sorted(pool)</font></div></div><div><div><font face="Courier">    size = len(copy)</font></div></div><div><div><font face="Courier">    if size % 2 == 1:</font></div></div><div><div><font face="Courier">        return copy[(size - 1) / 2]</font></div></div><div><div><font face="Courier">    else:</font></div></div><div><div><font face="Courier">        return (copy[size/2 - 1] + copy[size/2]) / 2</font></div></div><div><br></div></blockquote><div><font face="Monaco"><span style="font-size:11px">From my understanding of basic blocks in compilers, the above code snippet should have at least 3 basic blocks as follows:</span></font></div><div><font face="Monaco"><span style="font-size:11px"><span style="white-space:pre-wrap">  </span>1. Block 1 - all instructions up to and including those for the if test.</span></font></div><div><font face="Monaco"><span style="font-size:11px"><span style="white-space:pre-wrap">      </span>2. Block 2 - all instructions for the if body i.e the first return statement.</span></font></div><div><font face="Monaco"><span style="font-size:11px"><span style="white-space:pre-wrap"> </span>3. Block 3 - instructions for the else block i.e. the second return statement.</span></font></div><div><font face="Monaco"><span style="font-size:11px"><br></span></font></div><div><font face="Monaco"><span style="font-size:11px">My understanding of the the section on Control flow Graphs in the “Design of the CPython Compiler” also alludes to this - </span></font></div><div><font face="Monaco"><span style="font-size:11px"> </span></font></div><div><blockquote type="cite"><blockquote type="cite"><span style="color:rgb(62,67,73);line-height:21.6px;background-color:rgb(255,255,255)"><font face="Courier" size="1">As an example, consider an ‘if’ statement with an ‘else’ block. The guard on the ‘if’ is a basic block which is pointed to by the basic block containing the code leading to the ‘if’ statement. The ‘if’ statement block contains jumps (which are exit points) to the true body of the ‘if’ and the ‘else’ body (which may be NULL), each of which are their own basic blocks. Both of those blocks in turn point to the basic block representing the code following the entire ‘if’ statement.</font></span></blockquote></blockquote></div><div><br></div><div><font face="Monaco" style="font-size:11px">The CPython compiler however seems to generate 2 basic blocks for the above snippets - </font></div><div><font face="Monaco" style="font-size:11px"><span style="white-space:pre-wrap"> </span>1. Block 1 - all instructions up to and including the if statement and the body of the if statement (the first return statement in this case)</font></div><div><font face="Monaco" style="font-size:11px"><span style="white-space:pre-wrap">  </span>2. Block 2 - instructions for the else block (the second return statement)</font></div><div><font face="Monaco" style="font-size:11px"><br></font></div><div><font face="Monaco" style="font-size:11px">Is there any reason for this or have I somehow missed something in the CFG generation process?</font></div></div></blockquote><div><br></div><div>I have not looked at the code or the CFGs that are generated from your example code, but my guess is it's two blocks instead of three is because two block is all that's necessary to generate jump targets (since there's only one jump in that function). Since Python doesn't have block-level scope there's no need to generate a separate block in the `if` statement. And since the `if` statement will have a jump to the `else` block and otherwise fall through then there's only a need to have the block that starts the function and then the second block that the `if` jump goes to and that's it.</div></div></div>