[PATCH] Fix ARM unwind opcode assembler in several cases.

Logan Chien tzuhsiang.chien at gmail.com
Fri May 3 03:34:07 PDT 2013


Hi asl,

Changes to ARM unwind opcode assembler:

* Fix multiple .save or .vsave directives.  Besides, the
  order is preserved now.

* For the directives which will generate multiple opcodes,
  such as ".save {r0-r11}", the order of the unwind opcode
  is fixed now, i.e. the registers with less encoding value
  are popped first.

* Fix the $sp offset calculation.  Now, we can use the
  .setfp, .pad, .save, and .vsave directives at any order.

Changes to test cases:

* Add test case to check the order of multiple opcodes
  for the .save directive.

* Fix the incorrect $sp offset in the test case.  The
  stack pointer offset specified in the test case was
  incorrect.

* The opcode to restore $sp are slightly reordered.  The
  behavior are not changed, and the new output is same
  as the output of GNU as.  (File: eh-directive-pad.s,
  eh-directive-setfp.s)

http://llvm-reviews.chandlerc.com/D739

Files:
  lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp
  lib/Target/ARM/MCTargetDesc/ARMUnwindOpAsm.cpp
  lib/Target/ARM/MCTargetDesc/ARMUnwindOpAsm.h
  test/CodeGen/ARM/ehabi-mc-section.ll
  test/CodeGen/ARM/ehabi-mc.ll
  test/MC/ARM/eh-directive-multiple-offsets.s
  test/MC/ARM/eh-directive-pad.s
  test/MC/ARM/eh-directive-save.s
  test/MC/ARM/eh-directive-setfp.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D739.1.patch
Type: text/x-patch
Size: 26721 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130503/0a3f92bd/attachment.bin>


More information about the llvm-commits mailing list