<p dir="ltr">@Alex: thanks! If I don't get a review this week, I'll put it up :)</p>
<p dir="ltr">@Eric: gentle ping. I replied to the questions you asked. Are there any other concerns? </p>
<p dir="ltr">Thanks,<br>
Siddharth<br>
</p>
<br><div class="gmail_quote"><div dir="ltr">On Tue 21 Nov, 2017, 14:40 Alex Bradbury, <<a href="mailto:asb@asbradbury.org">asb@asbradbury.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 20 November 2017 at 10:32, Siddharth Bhat via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> Hello all,<br>
><br>
> I recently felt the need to have debug info at the LLVM IR level (because my<br>
> frontend's semantics vastly differ from IR that it doesn't make much sense<br>
> to use it). Asm can sometimes be too "low level", so it's nice to have the<br>
> ability to single-step at the IR level.<br>
><br>
> The pass that used to do this (-debugir) bitrotted and was removed from<br>
> tree. I spent the weekend bringing it back: <a href="https://reviews.llvm.org/D40207" rel="noreferrer" target="_blank">https://reviews.llvm.org/D40207</a><br>
><br>
> I'd love to have some help with reviews and tests I should write for this,<br>
> because I don't understand the debug info side of LLVM very well.<br>
><br>
> Also, is there a protocol to have a review be put on the "review corner"?<br>
<br>
Hi Siddharth, this is documented at <a href="http://llvmweekly.org/reviewcorner" rel="noreferrer" target="_blank">http://llvmweekly.org/reviewcorner</a><br>
which is linked to in each issue. Pasting the guidance here for<br>
convenience:<br>
<br>
"""<br>
For inclusion, the following should be true:<br>
* Your patch has gone at least two weeks without substantial review<br>
feedback OR this is your first patch to an LLVM project<br>
* You have updated the patch based on any review comments received so far<br>
* The patch has been compile tested against the current HEAD of the<br>
appropriate LLVM project, and rebased if necessary<br>
* You are willing to write a short two-sentence summary that explains<br>
1) what the patch does, and 2) why people might interested.<br>
"""<br>
<br>
Best,<br>
<br>
Alex<br>
</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>