[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