<br><br><div class="gmail_quote">On Mon Nov 24 2014 at 9:08:36 AM David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Honestly, debugir seemed like it went in half-baked and has rotted a bit since then. I'd probably advocate for removing it at this point - someone can resurrect it when they have time/care to more fully test it, etc. (the fact that you can remove this code without tests failing (I assume you tested that) is an indication that it's not well tested, etc)<br><div class="gmail_extra"><br></div></div></blockquote><div><br></div><div>It was in a little better shape than that when it went in. It had some basic functionality and someone to maintain it... and then their project was canceled :\</div><div><br></div><div>That said, I'm also ok with removing it.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Nov 23, 2014 at 7:26 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">While removing instances of `MDNode::replaceAllUsesWith()`, I came<br>
across this unlikely-to-be-correct RAUW of a compile unit.<br>
<br>
There are no tests that the updated debug info works correctly and it<br>
looks unlikely to be useful in any way -- are you debugging the original<br>
source, or the IR?  Instead, just delete the old debug info before<br>
creating something new.<br></blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br>I can imagine this could be useful - it's a slightly dodgy way to propagate flags down from a frontend into the debugir. "Optimized" for example, is meaningful even to debugIR (& the value that Clang sets would be generally correct - it does cause LLVM to run more optimizations, etc). SplitDebugFilename would also be relevant - if your frontend has enabled fission, not propagating this attribute would produce a weird result (the frontend would split out the .dwo sections, but they would be empty - at best this would produce a very uninteresting .dwo file, at worst, objdump might fail because there were no sections to move out).<br><br>But *shrug* Really dunno what to do with all this code.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div>