<div dir="ltr"><div><div><div class="inbox-inbox-F3hlO"><div dir="ltr"><div class="gmail_quote"><div>> I
 removed the pass because it wasn't maintained and the cost of keeping 
it up to date was more than the benefit, so a few questions:</div><div><br></div><div>> a) Are you planning on maintaining this?</div><div>Yes, I do plan on maintaining this.<br><br><br>> b) Are you going to be doing active work on it?<br></div><div>I do plan on writing additional tests, and maybe adding support for variables as described in the kaleidoscope docs, yes. <br><br><br></div><div>> c) What's the benefit here? What use case?<br></div><div>At least for me, the use case is when you have a miscompile from your frontend. It's easier to think about things on a LLVM IR level than on the assembly (at least for me). So, I prefer to have the ability to step through the LLVM IR in a debugger. <br><br>> Are you finding it hard to get a review on the patch?</div></div></div></div>Yes, I am.<br><br></div>Thanks a lot,<br></div>~Siddharth<br></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 20 Nov 2017 at 23:57 Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br></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_quote"><div dir="ltr">On Mon, Nov 20, 2017 at 2:32 AM Siddharth Bhat via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello all,<div><br></div><div>I recently felt the need to have debug info at the LLVM IR level (because my frontend's semantics vastly differ from IR that it doesn't make much sense to use it). Asm can sometimes be too "low level", so it's nice to have the ability to single-step at the IR level.</div><div><br></div><div>The pass that used to do this (-debugir) bitrotted and was removed from tree. I spent the weekend bringing it back: <a href="https://reviews.llvm.org/D40207" target="_blank">https://reviews.llvm.org/D40207</a></div><div><br></div></div></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>I removed the pass because it wasn't maintained and the cost of keeping it up to date was more than the benefit, so a few questions:</div><div><br></div><div>a) Are you planning on maintaining this?</div><div>b) Are you going to be doing active work on it?</div><div>c) What's the benefit here? What use case?</div></div></div><div dir="ltr"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div></div><div>I'd love to have some help with reviews and tests I should write for this, because I don't understand the debug info side of LLVM very well.</div><div><br></div><div>Also, is there a protocol to have a review be put on the "review corner"? I'd be glad if this patch could be listed there :) </div><div><br></div></div></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>Are you finding it hard to get a review on the patch?</div></div></div><div dir="ltr"><div class="gmail_quote"><div><br></div><div>-eric </div></div></div></blockquote></div><div dir="ltr">-- <br></div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Sending this from my phone, please excuse any typos!</div></div>