[ARM] Remove references to redundant cortex-a9-mp CPU

[• chatur01]

Summary

This CPU definition is redundant and was recently removed from LLVM (see http://reviews.llvm.org/D6057)

This patch removes the remaining references to it from Clang.

[rengolin]

I can't remember now, does that mean that "-mcpu=cortex-a9-mp" will work and mean "arm7tdmi" or will it fail with "unknown CPU"? I'm actually surprised that we had no tests for it. :)

If it fails, LGTM. If not, the odd folks using it may have a nasty selection of compile/link/run-time errors.

cheers,
--renato

[• chatur01]

In D6118#4, @rengolin wrote:

I can't remember now, does that mean that "-mcpu=cortex-a9-mp" will work and mean "arm7tdmi" or will it fail with "unknown CPU"?

It does fail,

$ clang -target arm-none-eabi -mcpu=cortex-a9-mp test.ll 
error: unknown target CPU 'cortex-a9-mp'

I'm actually surprised that we had no tests for it. :)

So was I! :)

Thank you for reviewing,
Charlie.

[• chatur01]

Hi Renato,

Thanks again for looking. Committed as r221458.

Charlie.

lib/Basic/Targets.cpp

     StringRef ArchName = getTriple().getArchName();
     if (CPU == "arm1136jf-s" || CPU == "arm1176jzf-s" || CPU == "mpcore")
       Features["vfp2"] = true;
-    else if (CPU == "cortex-a8" || CPU == "cortex-a9" ||
-             CPU == "cortex-a9-mp") {
+    else if (CPU == "cortex-a8" || CPU == "cortex-a9") {
       Features["vfp3"] = true;
       Features["neon"] = true;
     }

       .Cases("arm1176jz-s", "arm1176jzf-s", "6ZK")
       .Cases("arm1136jf-s", "mpcorenovfp", "mpcore", "6K")
       .Cases("arm1156t2-s", "arm1156t2f-s", "6T2")
-      .Cases("cortex-a5", "cortex-a7", "cortex-a8", "cortex-a9-mp", "7A")
+      .Cases("cortex-a5", "cortex-a7", "cortex-a8", "7A")
       .Cases("cortex-a9", "cortex-a12", "cortex-a15", "cortex-a17", "krait", "7A")
       .Cases("cortex-r4", "cortex-r5", "7R")
       .Case("swift", "7S")

lib/Driver/ToolChains.cpp

     .Case("xscale", "xscale")
     .Cases("arm1136j-s", "arm1136jf-s", "arm1176jz-s", "arm1176jzf-s", "armv6")
     .Case("cortex-m0", "armv6m")
-    .Cases("cortex-a5", "cortex-a7", "cortex-a8", "cortex-a9-mp", "armv7")
+    .Cases("cortex-a5", "cortex-a7", "cortex-a8", "armv7")
     .Cases("cortex-a9", "cortex-a12", "cortex-a15", "cortex-a17", "krait", "armv7")
     .Cases("cortex-r4", "cortex-r5", "armv7r")
     .Case("cortex-m3", "armv7m")

lib/Driver/Tools.cpp

     .Cases("arm1136j-s",  "arm1136jf-s",  "arm1176jz-s", "v6")
     .Cases("arm1176jzf-s",  "mpcorenovfp",  "mpcore", "v6")
     .Cases("arm1156t2-s",  "arm1156t2f-s", "v6t2")
-    .Cases("cortex-a5", "cortex-a7", "cortex-a8", "cortex-a9-mp", "v7")
+    .Cases("cortex-a5", "cortex-a7", "cortex-a8", "v7")
     .Cases("cortex-a9", "cortex-a12", "cortex-a15", "cortex-a17", "krait", "v7")
     .Cases("cortex-r4", "cortex-r5", "v7r")
     .Case("cortex-m0", "v6m")