[PATCH] Remove the cortex-a9-mp CPU.

Charlie Turner charlie.turner at arm.com
Fri Oct 31 08:28:19 PDT 2014


This CPU definition is redundant. The Cortex-A9 is defined as supporting multiprocessing extensions. Remove its definition and update appropriate tests.

LLVM defines both a `cortex-a9` CPU and a `cortex-a9-mp` CPU. The only difference between the two CPU definitions in `ARM.td` is that cortex-a9-mp contains the feature `FeatureMP` for multiprocessing extensions.

This is redundant since the Cortex-A9 is defined as having multiprocessing extensions in the TRMs.

The following thread from last year has a discussion about the possibility of removing this CPU,

http://lists.cs.uiuc.edu/pipermail/cfe-dev/2013-November/033201.html

It appears as though the pseudo-architectures were removed, but the `cortex-a9-mp` CPU was not. Clang currently has no tests that depend on this CPU being in LLVM, so this patch will not break Clang tests, but I will prepare a separate patch for Clang to remove its references to this CPU name.

http://reviews.llvm.org/D6057

Files:
  lib/Target/ARM/ARM.td
  test/CodeGen/ARM/build-attributes.ll
  test/CodeGen/ARM/prefetch.ll
  test/MC/ARM/arm_instructions.s
  test/MC/Disassembler/ARM/arm-tests.txt
  test/MC/Disassembler/ARM/thumb-tests.txt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6057.15622.patch
Type: text/x-patch
Size: 5541 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141031/c3b0623f/attachment.bin>


More information about the llvm-commits mailing list