Change ARM VFP VLDM/VSTM instructions to use addressing mode #4, just like
authorBob Wilson <bob.wilson@apple.com>
Fri, 27 Aug 2010 23:18:17 +0000 (23:18 +0000)
committerBob Wilson <bob.wilson@apple.com>
Fri, 27 Aug 2010 23:18:17 +0000 (23:18 +0000)
commitd4bfd54ec2947e73ab152c3c548e4dd4beb700ba
treed4f7a8392f6a8887699db0d8674914cd21d45782
parent11493aa47114c1992a195c3b9f9902cff11d1542
Change ARM VFP VLDM/VSTM instructions to use addressing mode #4, just like
all the other LDM/STM instructions.  This fixes asm printer crashes when
compiling with -O0.  I've changed one of the NEON tests (vst3.ll) to run
with -O0 to check this in the future.

Prior to this change VLDM/VSTM used addressing mode #5, but not really.
The offset field was used to hold a count of the number of registers being
loaded or stored, and the AM5 opcode field was expanded to specify the IA
or DB mode, instead of the standard ADD/SUB specifier.  Much of the backend
was not aware of these special cases.  The crashes occured when rewriting
a frameindex caused the AM5 offset field to be changed so that it did not
have a valid submode.  I don't know exactly what changed to expose this now.
Maybe we've never done much with -O0 and NEON.  Regardless, there's no longer
any reason to keep a count of the VLDM/VSTM registers, so we can use
addressing mode #4 and clean things up in a lot of places.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@112322 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/ARM/ARMAddressingModes.h
lib/Target/ARM/ARMAsmPrinter.cpp
lib/Target/ARM/ARMBaseInstrInfo.cpp
lib/Target/ARM/ARMCodeEmitter.cpp
lib/Target/ARM/ARMISelDAGToDAG.cpp
lib/Target/ARM/ARMInstrNEON.td
lib/Target/ARM/ARMInstrVFP.td
lib/Target/ARM/ARMLoadStoreOptimizer.cpp
lib/Target/ARM/AsmPrinter/ARMInstPrinter.cpp
lib/Target/ARM/Disassembler/ARMDisassemblerCore.cpp
test/CodeGen/ARM/vst3.ll