<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Jun 19, 2018 at 7:20 PM <<a href="mailto:paul.robinson@sony.com">paul.robinson@sony.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="EN-US">
<div class="gmail-m_3930622354721873407WordSection1">
<p class="MsoNormal" style="margin-left:0.5in">Our users really wanted the line table entry for line 2 to include the constant materialization code for some VS debugger feature.<u></u><u></u></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><a name="m_3930622354721873407__MailEndCompose"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">Same here.  I have serious doubts that the fastisel tactic of bubbling constants up to the top of the block really helps code size
 or performance that much, and it distinctly hurts the debugging experience.  It's reordering code at –O0 and that is generally a Bad Thing™.<u></u><u></u></span></a></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">--paulr</span></p></div></div></blockquote><div><br></div><div>Well, I hope I fixed that in FastISel in r327581, r331087, and r331088. Vedant is talking about SDISel.</div></div></div>