<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 9, 2016 at 10:32 AM Adrian Prantl <<a href="mailto:aprantl@apple.com">aprantl@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aprantl added a comment.<br class="gmail_msg">
<br class="gmail_msg">
I would still prefer adding an "mergeDebugLoc" API, even if it only returns an empty DebugLoc, so we:<br class="gmail_msg">
<br class="gmail_msg">
- make it explicit that the debug location is being intentionally dropped and that this code has been audited for correctly updating debug locations (otherwise the code looks no different from a pass that is just not handling debug info at all)<br class="gmail_msg">
- can improve on it later by, e.g., keeping identical DebugLocs<br class="gmail_msg"></blockquote><div><br></div><div>Sort of agree, though I'm not sure that a mergeDebugLoc that does nothing would be less confusing than the code that just omits it... (might give readers the mistaken impression that something's actually happening here, when it isn't)... so I'm sort of on the fence.<br><br>- Dave</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">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26256" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26256</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div>