<div dir="ltr">Hello all,<div><br></div><div>I am interested in this work, and have tested it a bit so far. In the latest revision, it seems to me that we do not have support in Clang (so it doesn't generate llvm.dbg.label intrinsic) and in Dwarf debug emission (so the backend doesn't generate the DW_TAG_label DIE from the DBG_LABEL Machine Instruction) yet. I attempted to implement these features, do you want me to submit the patches?</div><div><br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Son Tuan Vu</div></div></div>
<br><div class="gmail_quote">On Mon, Apr 2, 2018 at 5:37 PM, Adrian Prantl via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
> On Apr 1, 2018, at 9:12 AM, Hsiangkai Wang <<a href="mailto:hsiangkai@gmail.com">hsiangkai@gmail.com</a>> wrote:<br>
> <br>
> Hi all,<br>
> <br>
> I am sorry that I didn’t carefully think about how to handle labels in<br>
> inlined function.<br>
<br>
</span>There is no need to apologize! Thank you very much for engaging in the discussion and for contributing you patches.<br>
<span class="">> <br>
> Today, I did some simple experiments and found that some basic block<br>
> may be removed in CFGSimplifyPass and the attached metadata will be<br>
> eliminated. So, if the optimization is turned on, label metadata will<br>
> disappear. However, intrinsic will keep existing in the function if we<br>
> handle it correctly. So, if we take care about optimization and<br>
> inlining, to keep label metadata through intrinsic could be right for<br>
> free.<br>
> <br>
> I learned a lot from discussions. Thanks for your comments and<br>
> suggestions. If there is no other concern, I will keep my original<br>
> implementation and send patches to Phabricator again.<br>
<br>
</span>That sounds like a good plan.<br>
<br>
-- adrian<br>
<div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
</div></div></blockquote></div><br></div>