[PATCH] D17090: Fix LLVM's handling and detection of skylake CPUs

[Demikhovsky, Elena via llvm-commits]

We have
def : CannonlakeProc<"cannonlake">;
def : CannonlakeProc<"cnl">;

But "skl" is replaced with "skylake".
 Should we remove "cnl" as well?

-  Elena

-----Original Message-----
From: David Kreitzer [mailto:david.l.kreitzer at intel.com] 
Sent: Saturday, February 13, 2016 11:55
To: sanjoy at playingwithpointers.com; craig.topper at gmail.com; Demikhovsky, Elena <elena.demikhovsky at intel.com>
Cc: spatel at rotateright.com; Kreitzer, David L <david.l.kreitzer at intel.com>; mcrosier at codeaurora.org; llvm-commits at lists.llvm.org
Subject: Re: [PATCH] D17090: Fix LLVM's handling and detection of skylake CPUs

DavidKreitzer added a comment.

Thanks for doing that, Sanjoy! Yes, that sounds like a good plan. And these changes all look good to me.

Elena, are there any other places we need to change to match?


http://reviews.llvm.org/D17090



---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.