<div dir="ltr"><div>Ideally, yes, but I get the feeling that such a list would never be entirely complete, and would get out-of-date pretty quickly. :/ (It's also sort of unclear what 'familiar' should mean -- "have touched file $X"? "have made many significant contributions to pass $Y", "can LGTM piece $Z of file $A with confidence," ...?)</div><div><br></div><div>Generally, I just bug a code owner (from CODE_OWNERS.TXT) if I'm really unsure who should get a patch. They're often quite busy, so a response might not be instant, but between that and blame, finding someone shouldn't be too terrible. :)</div><div><br></div><div>If you're not on the IRC, you can also ask there. Sometimes they'll just say "blame the file," but many active contributors who know the project well are somewhat active there.</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, May 17, 2018 at 4:25 PM Dávid Bolvanský <<a href="mailto:david.bolvansky@gmail.com" target="_blank">david.bolvansky@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">Maybe a simple list of people familiar with certain parts would be helpful for new people interested in LLVM contributing? :) We have git blame but... many people who changed the lines in the certain file cannot do a real review in that part.</div><div class="gmail_extra"><br><div class="gmail_quote">2018-05-18 0:21 GMT+02:00 George Burgess IV <span dir="ltr"><<a href="mailto:george.burgess.iv@gmail.com" target="_blank">george.burgess.iv@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">No worries. LLVM is a large project with not-so-well-defined ownership. If someone jumped on my change and LGTM'ed it, I probably would've done the same thing. :)</div><br><div class="gmail_quote"><div><div class="m_-2996820071787500621m_1403676960927761441h5"><div dir="ltr">On Thu, May 17, 2018 at 3:06 PM Dávid Bolvanský via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_-2996820071787500621m_1403676960927761441h5"><div dir="auto"><div><br>Alright. I dont know persons who can review AA properly so I didnt know that rja is not familiar with AA.</div><div dir="auto"><br></div><div dir="auto"><br><div class="gmail_quote" dir="auto"><div dir="ltr">Dňa pi 18. 5. 2018, 0:02 Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>> napísal(a):<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, May 17, 2018 at 1:23 PM, David Bolvansky via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org" rel="noreferrer" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> Author: xbolva00<br>
> Date: Thu May 17 13:23:33 2018<br>
> New Revision: 332657<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=332657&view=rev" rel="noreferrer noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=332657&view=rev</a><br>
> Log:<br>
> [AA] cfl-anders-aa with field sensitivity<br>
><br>
> Summary:<br>
> There was some unfinished work started for offset tracking in CFLGraph by the author of implementation of Andersen algorithm. This work was completed and support for field sensitivity was added to the core of Andersen algorithm.<br>
><br>
> The performance results seem promising.<br>
><br>
> SPEC2006 int_base score was increased by 1.1 % (I  compared clang 6.0 with clang 6.0 with this patch). The avergae compile time was increased by +- 1 % according my measures with small and medium C/C++ projects (I did not tested it on the large projects with milions of lines of code)<br>
><br>
> Reviewers: chandlerc, george.burgess.iv, rja<br>
><br>
> Reviewed By: rja<br>
><br>
<br>
This change hasn't been reviewed by anybody familiar with the code. I<br>
would've given George at least a chance to take a look at this.<br>
<br>
--<br>
Davide<br>
</blockquote></div></div></div></div></div>
_______________________________________________<span><br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</span></blockquote></div>
</blockquote></div><br></div>
</blockquote></div>