[PATCH] D20988: [cpu-detection] Substantial refactor of Host CPU detection code (x86)

[Craig Topper via llvm-commits]

craig.topper added a comment.

Overall I think is fine, but I'm a little unsure about what the CPU type and subtype split is trying to accomplish. Particularly the COREI7 name seems confusing since i7 refers to the higher end Intel parts, but there are also i3/i5. And skylake-avx512 is actually a Xeon part and not marketed as an i7 at all.


http://reviews.llvm.org/D20988