[PATCH] [ARM] Improve handling of stack accesses in Thumb-1.

John Brawn john.brawn at arm.com
Fri Feb 13 07:45:25 PST 2015


While working on improving the spill estimation I realised that it wasn't that which was causing frame offsets to be considered invalid, but instead that ARMII::AddrModeT1_s was treated as having 5 bits of offset but it actually has 8 bits of offset (5 bits is true for FP-based loads, but FP-based loads aren't used in Thumb-1). I've attached a new patch that fixes that also.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7594

Files:
  lib/Target/ARM/ARMBaseRegisterInfo.cpp
  lib/Target/ARM/ARMISelDAGToDAG.cpp
  lib/Target/ARM/ARMInstrThumb.td
  test/CodeGen/Thumb/stack-access.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7594.19890.patch
Type: text/x-patch
Size: 5973 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150213/80ba013b/attachment.bin>


More information about the llvm-commits mailing list