[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