<div dir="ltr">Hey Abel, is there any chance you can add the patches to <a href="https://reviews.llvm.org/D5767">https://reviews.llvm.org/D5767</a> instead of starting a new review? Given the context on the other patch is rather extensive, that would help review :)<br><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 11, 2017 at 4:05 PM Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.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">Hi, thanks for letting me know, excited to see this make progress \o/<div><br></div><div>Generally, if you follow the guidelines here: <a href="https://llvm.org/docs/Phabricator.html" target="_blank">https://llvm.org/docs/Phabricator.html</a></div><div>you'll get a code review that is cc'ed to the cfe-commits list (you can also put me in as reviewer), which is generally how people find patches.</div><div><br></div><div>You can also continue the old review by manually specifying it when uploading the patch via arc or in the UI.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 11, 2017 at 1:16 PM Abel Sinkovics <<a href="mailto:abel@sinkovics.hu" target="_blank">abel@sinkovics.hu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Manuel,<br>
<br>
Kristóf (cc'd) has implemented the missing tests for the Templight pull<br>
request for Clang. I have updated the patch, but the system seems to<br>
have created another pull request as the result of this. You can find it<br>
here: <a href="https://reviews.llvm.org/D38818" rel="noreferrer" target="_blank">https://reviews.llvm.org/D38818</a><br>
<br>
Please let us know if there are further things we should address.<br>
<br>
Thanks,<br>
   Ábel<br>
<br>
<br>
</blockquote></div></blockquote></div></div>