[PATCH] X86 Haswell SchedModel: Fix wrong ports definition

Quentin Colombet qcolombet at apple.com
Wed Jan 29 10:32:56 PST 2014


Thanks for the quick review!

Committed revision 200403.

On Jan 29, 2014, at 9:46 AM, Nadav Rotem <nrotem at apple.com> wrote:

> LGTM.  Out of curiosity, does it change the performance of any of the workloads in the test suite?
I haven’t see any when I quickly checked yesterday. That said my machine was not really in a benchmarking configuration :).

-Quentin
> 
> 
> On Jan 29, 2014, at 9:42 AM, Quentin Colombet <qcolombet at apple.com> wrote:
> 
>> Hi Nadav,
>> 
>> Here is a patch that fixes a couple of types in the ports definition for the Haswell scheduling model.
>> 
>> Is it OK to commit?
>> 
>> Thanks,
>> -Quentin
>> <X86HaswellModel.patch>
>> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140129/00734660/attachment.html>


More information about the llvm-commits mailing list