<div dir="ltr">I started the llvm-dev thread for the instruction location tradeoffs (removing the location from instructions to improve profiling support - possibly at the cost of other debug info consumers).<br><br>The motivation for this flag is different - for added functionality that's generally (imho/based on conversations so far) pretty well justified for profile needs (the need for function start lines to make profiles somewhat resilient to unrelated code changes, etc - as mentioned in the review/patch description).</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jan 9, 2017 at 11:51 AM Eric Christopher via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">echristo added a comment.<br class="gmail_msg">
<br class="gmail_msg">
I don't think this ever was hashed out in the llvm-dev thread?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D25435" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25435</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>