[PATCH] [ARM] Use symbolic register names in .cfi directives only with IAS (PR19110)

Hans Wennborg hans at chromium.org
Wed Mar 12 10:26:36 PDT 2014


Hi compnerd,

This is a follow-up to r203635. Saleem pointed out that since symbolic register
names are much easier to read, it would be good if we could turn them off only
when we realle need to because we're using an external assembler.


It might seem more natural to make useDwarfRegNumForCFI() virtual, but since that gets called a bunch of times during asm emission, i went for setting the flag when changing IAS mode instead.

Please take a look!

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

Files:
  include/llvm/MC/MCAsmInfo.h
  lib/Target/ARM/MCTargetDesc/ARMMCAsmInfo.cpp
  lib/Target/ARM/MCTargetDesc/ARMMCAsmInfo.h
  test/CodeGen/ARM/debug-frame-large-stack.ll
  test/CodeGen/ARM/debug-frame-vararg.ll
  test/CodeGen/ARM/debug-frame.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3056.1.patch
Type: text/x-patch
Size: 24731 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140312/6ea44690/attachment.bin>


More information about the llvm-commits mailing list