<div dir="ltr">FWIW, this patch is fine.<div><br></div><div>Don't stress too much about this.</div><div><br></div><div>Of course, its nice to remind folks if they should wait for another review, but I think this patch is simple enough and low enough risk that anyone who understands CMake and can test it should feel fine providing an LGTM.</div>
<div><br></div><div>=]</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 7, 2014 at 3:40 AM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Am I reading it right that this patch was committed without a review by an approved reviewer (partly) because you weren't explicit enough?</p>

<div class="gmail_quote">07 янв. 2014 г. 3:51 пользователь "João Matos" <<a href="mailto:ripzonetriton@gmail.com" target="_blank">ripzonetriton@gmail.com</a>> написал:<div><div class="h5"><br type="attribution">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Hi Timur,<div><br></div><div>No, I am not an approved reviewer, I only commented the changes to give some feedback about those changes, and maybe bring some attention to this forgotten patch.  There's an explicit option to accept the revision in Phab which I did not use.</div>



</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 6, 2014 at 9:52 PM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Joao,<br>
<br>
Are you an approved reviewer for this part of the codebase?<br>
<br>
2014/1/6 João Matos <<a href="mailto:ripzonetriton@gmail.com" target="_blank">ripzonetriton@gmail.com</a>>:<br>
<div>><br>
>   LGTM.<br>
><br>
>   I've tested the patch and was able to complete a C++11 LLVM build with CMake on OSX.<br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D2381" target="_blank">http://llvm-reviews.chandlerc.com/D2381</a><br>
</div>> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>João Matos
</div>
</blockquote></div></div></div>
</blockquote></div><br></div>