<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Dec 13, 2013 at 1:45 PM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Dec 2, 2013, at 13:29, Manman Ren <<a href="mailto:manman.ren@gmail.com">manman.ren@gmail.com</a>> wrote:<br>

> Modified: llvm/trunk/lib/IR/DebugInfo.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/DebugInfo.cpp?rev=196158&r1=196157&r2=196158&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/DebugInfo.cpp?rev=196158&r1=196157&r2=196158&view=diff</a><br>

> ==============================================================================<br>
</div><div class="im">> --- llvm/trunk/lib/IR/AutoUpgrade.cpp (original)<br>
> +++ llvm/trunk/lib/IR/AutoUpgrade.cpp Mon Dec  2 15:29:56 2013<br>
</div><div class="im">> +/// Check the debug info version number, if it is out-dated, drop the debug<br>
> +/// info. Return true if module is modified.<br>
> +bool llvm::UpgradeDebugInfo(Module &M) {<br>
> +  if (getDebugInfoVersionFromModule(M) == DEBUG_METADATA_VERSION)<br>
> +    return false;<br>
> +<br>
> +  return StripDebugInfo(M);<br>
> +}<br>
><br>
</div>Hi Manman,<br>
<br>
sorry for commenting with such a delay.<br>
Can we somehow make this not just silently drop the debug info but print a warning or at least a DEBUG() message?<br>
Either here or in LLParser and BitCodeReader (which both seem to ignore the return value)?<br></blockquote><div><br></div><div>Hi Adrian,</div><div><br></div><div>We are going to use Quentin's work on "Add warning capabilities in LLVM" to emit warning messages.</div>
<div>Writing to errs() is not the right thing to do.</div><div><br></div><div>Thanks,</div><div>Manman</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
thanks!<br>
<span class="HOEnZb"><font color="#888888">  adrian<br>
</font></span></blockquote></div><br></div></div>