[PATCH] D46314: [X86][AMD][Bulldozer] Unbreak Bulldozer sub-type detection.

[Ganesh Gopalasubramanian via Phabricator via llvm-commits]

GGanesh added a comment.

gcc uses feature bits. That would be a differential ISA with respect to the previous gen. I think the idea is to get the ISA list as close to the underlying arch. When an older version of the compiler gets used which doesn't have the arch enabled, the compiler can fallback to the closest arch which enables the ISA list.

Model number for bulldozer should be as mentioned in the wikipedia as quoted by @craig.topper. 
For example, I have bdver3 machine with lscpu output as mentioned below.
<snip>
Core(s) per socket:    2
Socket(s):             1
NUMA node(s):          1
Vendor ID:             AuthenticAMD
CPU family:            21
Model:                 48
</snip>

So, checking for '3' in model for bdver3 will definitely fail this machine.


Repository:
  rL LLVM

https://reviews.llvm.org/D46314