[cpu-detection] [amdfam10] Return barcelona, and amdfam10 for all other subtypes. Address Bug 28067.

[asbirlea]

Summary

Along with the refactoring of Host.cpp, getHostCPUName() was modified to return more precise types for CPUs in amdfam10.
However, callers of getHostCPUName() do string matching on type, so this cannot be modified.
Currently there is support in the x86 backend for barcelona.
For all other subtypes the assumed return value is amdfam10.

Fix: getHostCPUName() returns barcelona subtype and amdfam10 for all others. This can be extended further when support for the other subtypes is added.

[echristo]

Can you elaborate on your summary? What's going on, why is this correct, what does it fix? Basically trying to make sure that the text committed says what's going on and why a change was made rather than what you're doing.

-eric

[echristo]

So, we already have support for barcelona, etc in the x86 backend. Perhaps we should just update the callers to remove the amdfam10- from the prefix in order to pass things down further? Take it as an alias to the cpu? Should it just return "barcelona" instead? (Will that end up causing incompatibility problems in compiler-rt? Might if we're not compatible that way).

-eric

[asbirlea]

There is support in the x86 backend for barcelona, gcc's options include it and all uses I found test for it.
For all other subtypes the assumed return value is amdfam10.
So, keep barcelona subtype and return amdfam10 for all others (this can be extended further when support for the other subtypes is added).

[echristo]

Seems reasonable for now. Can you update the summary (and commit message) accordingly please?

Thanks!

-eric

[echristo]

OK. Thanks!

-eric

llvm/trunk/lib/Support/Host.cpp

case AMDATHLON:
       case AMDATHLON_FX:
         return "athlon-fx";
       case AMDATHLON_64:
         return "athlon64";
       default:
         return "athlon";
       }
     case AMDFAM10H:
-      switch (Subtype) {
-      case AMDFAM10H_BARCELONA:
-        return "amdfam10-barcelona";
-      case AMDFAM10H_SHANGHAI:
-        return "amdfam10-shanghai";
-      case AMDFAM10H_ISTANBUL:
-        return "amdfam10-istanbul";
-      default:
         return "amdfam10";
-      }
     case AMDFAM14H:
       return "btver1";
     case AMDFAM15H:
       switch (Subtype) {
       case AMDFAM15H_BDVER1:
         return "bdver1";
       case AMDFAM15H_BDVER2:
         return "bdver2";