<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 1, 2010, at 2:58 PM, Chris Lattner wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 1, 2010, at 2:52 PM, Devang Patel wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 1, 2010, at 2:30 PM, Chris Lattner wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div><br>On Oct 1, 2010, at 12:00 PM, Devang Patel wrote:<br><br><blockquote type="cite">Author: dpatel<br></blockquote><blockquote type="cite">Date: Fri Oct 1 14:00:44 2010<br></blockquote><blockquote type="cite">New Revision: 115323<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">URL: <a href="http://llvm.org/viewvc/llvm-project?rev=115323&view=rev">http://llvm.org/viewvc/llvm-project?rev=115323&view=rev</a><br></blockquote><blockquote type="cite">Log:<br></blockquote><blockquote type="cite">Fix code gen crash reported in PR 8235. We still lose debug info for the unused argument here. This is a known limitation recorded debuginfo-tests/trunk/dbg-declare2.ll function 'f6' test case.<br></blockquote><br>Thanks for fixing the crash Devang. How hard is it to get the FI# here? It would be "really great" to be able to get debug info for these. Losing it at -O0 seems bad.<font class="Apple-style-span"><font class="Apple-style-span" color="#144FAE"><br></font></font></div></blockquote><br></div><div>We need to find a way to use SDNode created during LowerArugments() in such cases. This happens only when A) argument is not used at -O0 and B) we do not use FastISel. (In normal course, at -O0 we expect FastISel to handle entry block dbg.declares.) We had a scheme to save nodes for unused argument. IIRC, recent changes in the way we do bottom up fastisel, we broke that delicate link. Now, FuncInfo is cleared before dbg.declare from entry block is processed. We need to find another way. </div></div></blockquote><br></div><div>Why isn't there a frameindex assigned to it? Even if it is dead, there is no harm in creating the FI#</div></div></blockquote><br></div><div>That's what EmitFuncArgumentDbgValue() was expecting. That is a real bug.</div><div>-</div><div>Devang</div><br></body></html>