<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Apr 14, 2009, at 10:11 AM, Dan Gohman wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Verdana; font-size: 11px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; ">On Apr 13, 2009, at 1:05 PM, Devang Patel wrote:<br><br><blockquote type="cite"><br></blockquote><blockquote type="cite">On Apr 13, 2009, at 12:59 PM, Dan Gohman wrote:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">On Apr 10, 2009, at 5:17 PM, Devang Patel wrote:<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">Author: dpatel<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">Date: Fri Apr 10 19:17:54 2009<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">New Revision: 68848<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">URL:<span class="Apple-converted-space"> </span><a href="http://llvm.org/viewvc/llvm-project?rev=68848&view=rev">http://llvm.org/viewvc/llvm-project?rev=68848&view=rev</a><br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">Log:<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">If linkage name is not available then use function display name as<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">linkage name.<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">Hi Devang,<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">What does this fix? Why is a bogus linkage name better than no<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">linkage name?<br></blockquote></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">In C, linkage name is same as name.<br></blockquote><br>And in pretty much everything else, it can be different.</span></blockquote><div><br></div>And if it is different then gdb wants to know it. If linkage name is different then the FE should have found the linkage name. </div><div><br><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Verdana; font-size: 11px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; "><blockquote type="cite"><span class="Apple-style-span" style="-webkit-text-stroke-width: -1; ">This info is used during FastISel to decide whether we are at the</span></blockquote><blockquote type="cite">beginning of inlined subroutine or not.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"> std::string SPName;<br></blockquote><blockquote type="cite"> Subprogram.getLinkageName(SPName);<br></blockquote><blockquote type="cite"> if (!SPName.empty()<br></blockquote><blockquote type="cite"> && strcmp(SPName.c_str(), MF.getFunction()-<br></blockquote><blockquote type="cite"><blockquote type="cite">getNameStart())) {<br></blockquote></blockquote><br>Could this use<br><br> Subprogram.getGV() == MF.getFunction()<br><br>instead?</span></blockquote><div><br></div>It won't work. The Subprogram does not have actual Fn. It only carries information for function named "blah".</div><div><br></div><div>-</div><div>Devang</div><div><br></div></body></html>