[PATCH] Enable EHABI by default on non-Darwin ARM targets

Renato Golin renato.golin at linaro.org
Mon Jan 27 08:26:06 PST 2014


Hi asl,

After all hard work to implement the EHABI and with the test-suite passing, it's time to turn it on by default and allow users to disable it as a work-around while we fix the eventual bugs that show up.

This commit also remove the -arm-enable-ehabi-descriptors, since we want the tables to be printed every time the EHABI is turned on for non-Darwin ARM targets.

Current EH test in the test-suite pass with these changes, so a follow-
up commit on the test-suite will re-enable them.

The only problem is that MCJIT can't handle some new relocation, so would be good to disable EHABI while JIT-ing, but I have no idea how. Most (all?) MCJIT tests fail with this commit, so we need to either disable that or fix it. I'd rather disable to get to work on it in separate, since static compilation is working well.

  Not implemented relocation type!
  UNREACHABLE executed at /home/user/devel/llvm/src/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp:482!


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

Files:
  lib/CodeGen/AsmPrinter/ARMException.cpp
  lib/Target/ARM/ARMAsmPrinter.cpp
  lib/Target/ARM/MCTargetDesc/ARMMCAsmInfo.cpp
  test/CodeGen/ARM/arm-ttype-target2.ll
  test/CodeGen/ARM/ehabi-filters.ll
  test/CodeGen/ARM/ehabi-no-landingpad.ll
  test/CodeGen/ARM/ehabi-unwind.ll
  test/CodeGen/ARM/ehabi.ll
  test/CodeGen/ARM/setcc-sentinals.ll
  test/CodeGen/Thumb2/constant-islands.ll
  test/MC/ARM/data-in-code.ll
  test/MC/ARM/elf-thumbfunc-reloc.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2627.1.patch
Type: text/x-patch
Size: 9719 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140127/a2a1eb9d/attachment.bin>


More information about the llvm-commits mailing list