<div dir="ltr">I would be happy to help. Could you send me the example patch? Where would I submit my patch to be reviewed?<br><br>Thanks,<br>Henry</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 11, 2018 at 8:31 PM, Dean Michael Berris <span dir="ltr"><<a href="mailto:dean.berris@gmail.com" target="_blank">dean.berris@gmail.com</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 12 Jun 2018, at 07:49, Henry Zhu via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br>
> <br>
> Hello,<br>
> <br>
>  <br>
> I’ve noticed that XRay does not report the PID in the trace log. Could this be added for multi-process support? (Maybe as a command line argument).<br>
> <br>
<br>
</span>We can certainly add the PID in the trace header. This will be a format change which will require a version increment, but we’ve done that before so it should be doable.<br>
<br>
Do you have time to implement this feature? It will span LLVM and compiler-rt, but it should be straight-forward to do so.<br>
<br>
Let me know and I’ll get you an example patch that did this for some of the changes we’ve done recently in this regard. Keith can also help as he’s worked on updating the FDR format in the past as well.<br>
<br>
Cheers<br>
<span class="HOEnZb"><font color="#888888"><br>
-- Dean<br>
<br>
</font></span></blockquote></div><br></div>