[llvm-commits] FW: [llvm][PATCH][Review request] Add SSE itineraries for Atom Scheduling
Gurd, Preston
preston.gurd at intel.com
Mon Feb 27 07:16:19 PST 2012
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: atomsse.diff
Type: application/octet-stream
Size: 236876 bytes
Desc: atomsse.diff
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120227/5f0c9d6d/attachment.obj>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ATT00001.txt
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120227/5f0c9d6d/attachment.txt>
More information about the llvm-commits
mailing list