ARM: Set default CPU for OpenBSD/arm to Cortex-A8

[brad]

Summary

Set default CPU for OpenBSD/arm to Cortex-A8

[rengolin]

Tests?

[brad]

In D30413#687461, @rengolin wrote:

Tests?

I posted the diff here as I wasn't sure what to use as a test (as opposed to from Clang's perspective).

[t.p.northover]

You probably want unittests/ADT/TripleTest.cpp. A good way to find this out yourself is to "git blame" the file you're modifying and look at what tests were added for similar bits. A refactoring in this case, but it's still pretty evident from the diff.

[brad]

Add a test

[rengolin]

I'm not sure why OpenBSD needs "arm" to default to cortex-A8, but assuming that's the correct behaviour, LGTM. It's not like others don't do that as well. :)

lib/Support/Triple.cpp

       return "strongarm";
     }
   case llvm::Triple::NaCl:
+  case llvm::Triple::OpenBSD:
     return "cortex-a8";
   default:
     switch (getEnvironment()) {

unittests/ADT/TripleTest.cpp

     EXPECT_EQ("cortex-a8", Triple.getARMCPUForArch());
   }
   {
+    llvm::Triple Triple("arm--openbsd");
+    EXPECT_EQ("cortex-a8", Triple.getARMCPUForArch());
+  }
+  {
     llvm::Triple Triple("armv6-unknown-freebsd");
     EXPECT_EQ("arm1176jzf-s", Triple.getARMCPUForArch());
   }