[PATCH] Refactor ARM subarchitecture parsing.

[Renato Golin]

================
Comment at: lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.cpp:163
@@ +162,3 @@
+    break;
+  case Triple::v4t:
+    ARMArchFeature = "+v4t";
----------------
Gabor Ballabas wrote:
> Renato Golin wrote:
> > you could add the NoSubArch here, as a default.
> I tried to add it, but it made 12 tests fail:
> Failing Tests (12):
>     LLVM :: CodeGen/ARM/2010-03-18-ldm-rtrn.ll
>     LLVM :: CodeGen/ARM/2013-04-05-Small-ByVal-Structs-PR15293.ll
>     LLVM :: CodeGen/ARM/2014-02-21-byval-reg-split-alignment.ll
>     LLVM :: CodeGen/ARM/arguments.ll
>     LLVM :: CodeGen/ARM/armv4.ll
>     LLVM :: CodeGen/ARM/call_nolink.ll
>     LLVM :: CodeGen/ARM/debug-frame.ll
>     LLVM :: CodeGen/ARM/ehabi.ll
>     LLVM :: CodeGen/ARM/inlineasm-ldr-pseudo.ll
>     LLVM :: CodeGen/ARM/integer_insertelement.ll
>     LLVM :: CodeGen/ARM/jump_tables.ll
>     LLVM :: MC/ARM/arm-thumb-cpus.s
Interesting... Maybe the default sub-arch should be v4 not v4t.

http://reviews.llvm.org/D4375