[llvm] r248152 - [ARM] Handle +t2dsp feature as an ArchExtKind in ARMTargetParser.def

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 21 09:37:44 PDT 2015


Hi Artyom,

I have reverted this in r248174.

Cheers,

James

On Mon, 21 Sep 2015 at 08:58 James Molloy <james at jamesmolloy.co.uk> wrote:

> Hi Artyom,
>
> 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.
>
> 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.
>
> This commit and the other must be reverted as soon as possible, because
> you didn't follow the process that is explicit on llvm.org.
>
> If you can't revert it, I'll do the revert myself soon.
>
> Cheers,
>
> James
> On Mon, 21 Sep 2015 at 08:29, Artyom Skrobov <artyom.skrobov at arm.com>
> wrote:
>
>> James, your own message quotes me addressing Peter’s concerns:
>>
>> > Could you point me to the code review here? I can see a review
>> > thread but nowhere there does anyone approve the patch. In
>> > fact Peter raises concerns that seemingly aren't addressed.
>> >
>> > [skip]
>> >
>> >> Following review comments, also updating the description of
>> >> FeatureDSPThumb2in ARM.td.
>> >>
>> >> Differential Revision: http://reviews.llvm.org/D12937
>>
>> Hope this helps.
>>
>>
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150921/8cb9d26f/attachment.html>


More information about the llvm-commits mailing list