<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 19, 2017 at 11:12 AM Anmol P. Paralkar 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">appcs added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Thank you, everybody, for the valuable inputs.<br class="gmail_msg">
<br class="gmail_msg">
I understand the the codegen consistency [-g, -g0] requirement; I agree.<br class="gmail_msg">
<br class="gmail_msg">
Would it be fair to say that -mergefunc-preserve-debug-info is more suited to being placed under –Og?<br class="gmail_msg"></blockquote><div><br></div><div>There's still some hope that -Og may be only a compatibility flag & might be able to be implemented as an alias for -O1.<br><br>That said, if this particular feature's a really good example of something we really want to do for debugger use and that's the main/only reason to do it - that'd be a really good discussion to have. If the community can come to an agreement that that's true in this case we may be able to disprove that -Og/-O1 equivalence goal sooner rather than later.<br><br>All I mean to say is it may require some discussion to work/walk through all of this. (probably better had on llvm-dev or cfe-dev, rather than in a review thread)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="gmail_msg">
I did not follow the "embed as a function attribute" part; please could you elaborate?<br class="gmail_msg">
<br class="gmail_msg">
I will submit a subsequent patch (upon this current base patch) that implements an API to enable/disable this behaviour<br class="gmail_msg">
(-mergefunc-preserve-debug-info) in the near future. Would that be alright?<br class="gmail_msg">
<br class="gmail_msg">
Is the current base patch (Diff 84419) OK to be committed?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D28075" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28075</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div>