[PATCH] Move Post RA Scheduling flag bit into SchedMachineModel

hfinkel at anl.gov hfinkel at anl.gov
Sun Jul 13 09:21:19 PDT 2014


----- Original Message -----
> From: "Sanjay Patel" <spatel at rotateright.com>
> To: spatel at rotateright.com, nrotem at apple.com, atrick at apple.com, hfinkel at anl.gov
> Cc: "amara emerson" <amara.emerson at arm.com>, llvm-commits at cs.uiuc.edu
> Sent: Sunday, July 13, 2014 11:15:46 AM
> Subject: Re: [PATCH] Move Post RA Scheduling flag bit into SchedMachineModel
> 
> Not sure what the policy is on this - Hal, you already marked this
> with 'LGTM', but the patch has grown quite a bit since that.

Generally speaking, we ask contributors to use their best judgment as to whether the additional changes are obvious/mechanical-enough or not.

> Do you
> want to give this another look over?

Sure. I'll look at it again.

 -Hal

> 
> http://reviews.llvm.org/D4217
> 
> 
>

http://reviews.llvm.org/D4217






More information about the llvm-commits mailing list