[cfe-dev] Tests added to the Templight pull request

[Abel Sinkovics via cfe-dev]

Hi Manuel,

I've tried to do so, but the UI doesn't let me do that. I can choose 
between creating a new revision and updating the one I recently created 
(D38818). I assume this is because D5767 was created by Mikael (cc'd), 
not me.

Regards,
   Ábel


Am 2017-10-16 um 00:06 schrieb Manuel Klimek:
> Hey Abel, is there any chance you can add the patches to 
> https://reviews.llvm.org/D5767 instead of starting a new review? Given 
> the context on the other patch is rather extensive, that would help 
> review :)
>
> On Wed, Oct 11, 2017 at 4:05 PM Manuel Klimek <klimek at google.com 
> <mailto:klimek at google.com>> wrote:
>
>     Hi, thanks for letting me know, excited to see this make progress \o/
>
>     Generally, if you follow the guidelines here:
>     https://llvm.org/docs/Phabricator.html
>     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.
>
>     You can also continue the old review by manually specifying it
>     when uploading the patch via arc or in the UI.
>
>
>     On Wed, Oct 11, 2017 at 1:16 PM Abel Sinkovics <abel at sinkovics.hu
>     <mailto:abel at sinkovics.hu>> wrote:
>
>         Hi Manuel,
>
>         Kristóf (cc'd) has implemented the missing tests for the
>         Templight pull
>         request for Clang. I have updated the patch, but the system
>         seems to
>         have created another pull request as the result of this. You
>         can find it
>         here: https://reviews.llvm.org/D38818
>
>         Please let us know if there are further things we should address.
>
>         Thanks,
>            Ábel
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20171016/621ed964/attachment.html>