[llvm] d3917bb - [X86] Add title comment to separate the "CPU Families" features from the other subtarget features. NFCI.

[Simon Pilgrim via llvm-commits]

Author: Simon Pilgrim
Date: 2021-08-03T12:53:57+01:00
New Revision: d3917bbfc6bdd0a599a4d3350a4f215a7b33cf94

URL: https://github.com/llvm/llvm-project/commit/d3917bbfc6bdd0a599a4d3350a4f215a7b33cf94
DIFF: https://github.com/llvm/llvm-project/commit/d3917bbfc6bdd0a599a4d3350a4f215a7b33cf94.diff

LOG: [X86] Add title comment to separate the "CPU Families" features from the other subtarget features. NFCI.

Hopefully we can get rid of these some day...

Added: 
    

Modified: 
    llvm/lib/Target/X86/X86.td

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/X86/X86.td b/llvm/lib/Target/X86/X86.td
index 53bbd93798ac..97c147dd42fa 100644
--- a/llvm/lib/Target/X86/X86.td
+++ b/llvm/lib/Target/X86/X86.td
@@ -531,6 +531,11 @@ def FeatureUseGLMDivSqrtCosts
 def FeatureUseAA : SubtargetFeature<"use-aa", "UseAA", "true",
                                     "Use alias analysis during codegen">;
 
+//===----------------------------------------------------------------------===//
+// X86 CPU Families
+// TODO: Remove these - use general tuning features to determine codegen.
+//===----------------------------------------------------------------------===//
+
 // Bonnell
 def ProcIntelAtom : SubtargetFeature<"", "X86ProcFamily", "IntelAtom", "">;
 // Silvermont