[PATCH] D63018: [X86] Attempt to make the Intel core CPU inheritance a little more readable and maintainable

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 10 02:28:06 PDT 2019


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM - the goto is unfortunate but more maintainable and less confusing then the current state of things



================
Comment at: clang/lib/Basic/Targets/X86.cpp:157
+    setFeatureEnabledImpl(Features, "pku", true);
+    goto SkylakeCommon;
+
----------------
Worth adding a comment explaining the AVX512 ISA divergence?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63018/new/

https://reviews.llvm.org/D63018





More information about the cfe-commits mailing list