[X86] Disable CLWB in Cannon Lake

[GBuella]

Summary

Cannon Lake does not support CLWB, therefore it
does not include all features listed under SKX anymore.

Instead, enumerate all SKX features with the exception of CLWB.

[echristo]

Testcase?

[craig.topper]

This bug exists in clang too.

[craig.topper]

How do we test this? You'd have to verify that the clwb intrinsic throws a cannot select error on cannonlake, but do we do that in llc lit tests today?

[echristo]

We have some testcases like this, but not a lot. We could add one - also we should probably not assemble if we're using cannonlake as well. The aarch64 backend has some examples of llvm-mc tests in this case.

In D43380#1010794, @craig.topper wrote:

How do we test this? You'd have to verify that the clwb intrinsic throws a cannot select error on cannonlake, but do we do that in llc lit tests today?

[craig.topper]

I thought our assemble stance was that everything was available all the time. For example, the bug a few weeks back that said we should always parse xmm16-xmm31 registers regardless of avx512 enabling. Or are we saying we should support everything unlesss you pick a specific CPU?

[GBuella]

In D43380#1010794, @craig.topper wrote:

How do we test this? You'd have to verify that the clwb intrinsic throws a cannot select error on cannonlake, but do we do that in llc lit tests today?

I have not seen any such tests for CPU being selected in LLVM.
The closest thing I found is in clang ( test/Preprocessor/predefined-arch-macros.c ), I'm adding it there.

[GBuella]

See also: https://reviews.llvm.org/D43459

[craig.topper]

You can do something like test/CodeGen/X86/pr33772.ll

[GBuella]

Adding tests for checking that the
cpu flags affect clwb intrinsic usage as they
should.

[GBuella]

In D43380#1010942, @craig.topper wrote:

I thought our assemble stance was that everything was available all the time. For example, the bug a few weeks back that said we should always parse xmm16-xmm31 registers regardless of avx512 enabling. Or are we saying we should support everything unlesss you pick a specific CPU?

Yes, the MC assembler allows any instruction, so this patch has no relation to that.

[craig.topper]

LGTM

lib/Target/X86/X86.td

class SkylakeServerProc<string Name> : ProcModel<Name, SkylakeServerModel,
                                                  SKXFeatures.Value, [
   ProcIntelSKX,
   FeatureHasFastGather,
   FeaturePOPCNTFalseDeps
 ]>;
 def : SkylakeServerProc<"skylake-avx512">;
 def : SkylakeServerProc<"skx">; // Legacy alias.
 
-def CNLFeatures : ProcessorFeatures<SKXFeatures.Value, [
+def CNLFeatures : ProcessorFeatures<SKLFeatures.Value, [
+  FeatureAVX512,
+  FeatureCDI,
+  FeatureDQI,
+  FeatureBWI,
+  FeatureVLX,
+  FeaturePKU,
   FeatureVBMI,
   FeatureIFMA,
   FeatureSHA
 ]>;
 
 class CannonlakeProc<string Name> : ProcModel<Name, SkylakeServerModel,
                                               CNLFeatures.Value, [
   ProcIntelCNL,