<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Aug 31, 2017, at 7:09 AM, Sean Eveson via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" class="">llvm-dev@lists.llvm.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class=""><p class="MsoNormal"><font face="arial, helvetica, sans-serif" class=""><span style="" class="">Hi All,</span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font></p><div class=""><font face="arial, helvetica, sans-serif" class=""><span style="" class=""> </span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font><br class="webkit-block-placeholder"></div><p class="MsoNormal"><font face="arial, helvetica, sans-serif" class=""><span style="" class="">We have a local change in the
X86AsmPrinter that outputs a section containing metadata on function stack
sizes. We use this to measure changes to stack size between versions of the
compiler and it also allows our licensees to do the same for their code.</span></font></p></div></div></div></blockquote><div><br class=""></div><div>Have you considered using optimization remarks for this rather than rolling your own infrastructure. Francis (cc’ed) did something like this in <a href="https://reviews.llvm.org/D35645" class="">https://reviews.llvm.org/D35645</a>.</div><div><br class=""></div><div>Adam</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class=""><p class="MsoNormal"><font face="arial, helvetica, sans-serif" class=""><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font></p><div class=""><font face="arial, helvetica, sans-serif" class=""><span style="" class=""> </span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font><br class="webkit-block-placeholder"></div><p class="MsoNormal"><font face="arial, helvetica, sans-serif" class=""><span style="" class="">The section simply contains pairs
of function symbol references (8 byte) and stack sizes (unsigned LEB128).</span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font></p><div class=""><span style="color:rgb(33,33,33)" class=""><font face="arial, helvetica, sans-serif" class=""> <span class=""></span></font></span><br class="webkit-block-placeholder"></div><p class="MsoNormal"><font face="arial, helvetica, sans-serif" class=""><span style="" class="">We would like to upstream this
change as a PS4 only modification, or as a more general cross platform one.</span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font></p><div class=""><font face="arial, helvetica, sans-serif" class=""><span style="" class=""> </span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font><br class="webkit-block-placeholder"></div><p class="MsoNormal"><font face="arial, helvetica, sans-serif" class=""><span style="" class="">Would people be interested in (or
happy with) such a patch, target specific or otherwise?</span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font></p><div class=""><font face="arial, helvetica, sans-serif" class=""><span style="" class=""> </span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font><br class="webkit-block-placeholder"></div><p class="MsoNormal"><font face="arial, helvetica, sans-serif" class=""><span style="" class="">Thanks,</span><span style="color:rgb(33,33,33)" class=""><span class=""></span></span></font></p></div><font face="arial, helvetica, sans-serif" class=""><br clear="all" class=""></font><div class=""><div class="gmail_signature"><div dir="ltr" class=""><div class="gmail_signature"><div dir="ltr" class=""><font face="arial, helvetica, sans-serif" class=""><span style="line-height: 18.85px;" class="">Sean Eveson</span><br style="line-height: 18.85px;" class=""><span style="line-height: 18.85px;" class="">SN Systems - Sony Interactive Entertainment</span></font><br class=""></div></div></div></div></div>
</div>
_______________________________________________<br class="">LLVM Developers mailing list<br class=""><a href="mailto:llvm-dev@lists.llvm.org" class="">llvm-dev@lists.llvm.org</a><br class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev<br class=""></div></blockquote></div><br class=""></body></html>