<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi Eli, <div><br><div> The timer code LGTM.   Why do we need an additional command line flag ? <div><br></div><div>Nadav<br><div><br><div><div>On Mar 19, 2013, at 3:36 PM, Eli Bendersky <<a href="mailto:eliben@google.com">eliben@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;">Hello,<br><br>Following the discussion here -<br><a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2013-March/060095.html">http://lists.cs.uiuc.edu/pipermail/llvmdev/2013-March/060095.html</a><br><br>This patch adds an option -time-ir-parsing that measures the time it<br>took to parse IR (either from bitcode or from text assembly). This<br>should bring the total amount of time measured for compilation time by<br>LLVM itself closer to wallclock time.<br><br>Eli<br><br>P.S. The patch IMHO isn't very pretty because ParseIR is implemented<br>in a header file for some reason. I don't see the advantage of its<br>being there, but the disadvantages are many (for example one needs to<br>rebuild > 500 files for every source change).<br><span><time-ir-parsing.1.patch></span>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a></div></blockquote></div><br></div></div></div></div></body></html>