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

[Sanjoy Das via llvm-commits]

sanjoy added a comment.

In http://reviews.llvm.org/D17090#352306, @llvm-commits wrote:

> We have
>  def : CannonlakeProc<"cannonlake">;
>  def : CannonlakeProc<"cnl">;
>
> But "skl" is replaced with "skylake".
>
>   Should we remove "cnl" as well?


That makes sense (since in http://reviews.llvm.org/D16756 you use `"cannonlake"` as well).  I'll fix this tomorrow.

> 

> 

> - Elena



http://reviews.llvm.org/D17090