[PATCH] [ARM] Use DWARF register numbers for CFI directives in ELF assembly
Hans Wennborg
hans at chromium.org
Tue Mar 11 19:18:56 PDT 2014
================
Comment at: lib/Target/ARM/MCTargetDesc/ARMMCAsmInfo.cpp:59
@@ -57,1 +58,3 @@
+ // gas doesn't handle VFP register names in cfi directives.
+ DwarfRegNumForCFI = true;
}
----------------
Saleem Abdulrasool wrote:
> Would you mind basing this on whether or not we are using the integrated assembler? I think that the CFI is easier to consume (as a human) when the register names are used. If we are using an external assembler, we can fall back to the DWARF register numbers rather than the register names.
I guess we could do that. The only downside is that it adds another test dimension :/
http://llvm-reviews.chandlerc.com/D3049
More information about the llvm-commits
mailing list