<div dir="ltr">Hi Artyom,<div><br></div><div>I have reverted this in r248174.</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 21 Sep 2015 at 08:58 James Molloy <<a href="mailto:james@jamesmolloy.co.uk">james@jamesmolloy.co.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Artyom,<br><br>I'm afraid that's not how code review works. If you request code review, you must wait for that review to be approved before committing. You've raised two Phab reviews, which are requests for code review, and committed both without approval. <br><br>Peter had some concerns- the entire point of code review is that you address those concerns *and he accepts your changes*. Just saying you've addressed them doesn't mean you have to his satisfaction. Also, there may be more reviewers to chirp in - you didn't give them time to do so. <br><br>This commit and the other must be reverted as soon as possible, because you didn't follow the process that is explicit on <a href="http://llvm.org" target="_blank">llvm.org</a>. <br><br>If you can't revert it, I'll do the revert myself soon.<br><br>Cheers,<br><br>James<br><div class="gmail_quote"><div dir="ltr">On Mon, 21 Sep 2015 at 08:29, Artyom Skrobov <<a href="mailto:artyom.skrobov@arm.com" target="_blank">artyom.skrobov@arm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">James, your own message quotes me addressing Peter’s concerns:<br>
<br>
> Could you point me to the code review here? I can see a review<br>
> thread but nowhere there does anyone approve the patch. In<br>
> fact Peter raises concerns that seemingly aren't addressed.<br>
><br>
> [skip]<br>
><br>
>> Following review comments, also updating the description of<br>
>> FeatureDSPThumb2in ARM.td.<br>
>><br>
>> Differential Revision: <a href="http://reviews.llvm.org/D12937" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12937</a><br>
<br>
Hope this helps.<br>
<br>
<br>
<br>
</blockquote></div></blockquote></div>