[llvm-commits] [llvm][PATCH][Review request] Add SSE itineraries for Atom Scheduling

[Gurd, Preston]

Thanks Evan!

Although "make check" passes with my latest checkout (151543), I cannot commit the patch, because I do not yet have commit permission. Could you please commit the patch for me?

Thanks

Preston


-----Original Message-----
From: Evan Cheng [mailto:evan.cheng at apple.com] 
Sent: Monday, February 27, 2012 2:31 PM
To: Gurd, Preston
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] [llvm][PATCH][Review request] Add SSE itineraries for Atom Scheduling

LGTM. Please go ahead and commit the patch assuming all the tests pass.

Evan

On Feb 27, 2012, at 7:16 AM, Gurd, Preston wrote:

> Ping...
> 
> -----Original Message-----
> From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-bounces at cs.uiuc.edu] On Behalf Of Gurd, Preston
> Sent: Wednesday, February 22, 2012 8:14 PM
> To: Bill Wendling
> Cc: llvm-commits at cs.uiuc.edu
> Subject: Re: [llvm-commits] [llvm][PATCH][Review request] Add SSE itineraries for Atom Scheduling
> 
> Hello Bill,
> 
> Thank you for having a look at this patch.
> 
> The patch basically adds an itinerary class (whose name is prefixed by IIC_) or a group of itinerary classes (prefixed by SSE_) to every instruction emitted by X86InstrSSE.td, except for AVX only instructions or classes.
> 
> In my original patch, there a few changes to the formatting which came about because I undid a change without undoing the change in formatting. I have corrected this in the attached updated patch.
> 
> In pretty much all of the rest of the patch, it was often necessary to add a newline in the middle of the changed line in order to observe the 80 column width rule. If you would like me to provide a version of this patch which does not split lines, then please let me know and I will do so.
> 
> Thanks,
> 
> Preston
> 
> 
> -----Original Message-----
> From: Bill Wendling [mailto:wendling at apple.com] 
> Sent: Sunday, February 19, 2012 3:17 PM
> To: Gurd, Preston
> Cc: llvm-commits at cs.uiuc.edu
> Subject: Re: [llvm-commits] [llvm][PATCH][Review request] Add SSE itineraries for Atom Scheduling
> 
> Hi Preston,
> 
> It's hard to tease out your actual change from the formatting changes. Could you separate them please?
> 
> -bw
> 
> On Feb 17, 2012, at 1:33 PM, Gurd, Preston wrote:
> 
>> The attached patch adds instruction itineraries for the SSE instructions to the instruction scheduler for the Intel Atom.
>> 
>> Please review.
>> 
>> Thanks,
>> 
>> Preston
>> 
>> --
>> Preston Gurd <preston.gurd at intel.com>
>>  Intel Waterloo
>> 
>> 
>> 
>> <atomsse.diff>_______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 
> <atomsse.diff>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits