[PATCH] D45240: [ARM] Compute a target feature which corresponds to the ARM version.

[Florian Hahn via Phabricator via cfe-commits]

fhahn added inline comments.


================
Comment at: lib/Basic/Targets/ARM.cpp:345
   // get default FPU features
+  llvm::ARM::ArchKind Arch = llvm::ARM::parseArch(getTriple().getArchName());
   unsigned FPUKind = llvm::ARM::getDefaultFPU(CPU, Arch);
----------------
Is there a reason we re-compute the Arch from the triple here, but use the member variable ArchKind above? Shouldn't they both be the same ?


Repository:
  rC Clang

https://reviews.llvm.org/D45240