[PATCH] Emit DWARF info for all code section in an assembly file
Oliver Stannard
oliver.stannard at arm.com
Thu Mar 13 08:49:53 PDT 2014
> Let's split the testcase generalization out
Done, see new dependency
> I really don't want to add -gdwarf-N to llvm-mc
Why is that? I understand that we don't _have_ to maintain compatibility with clang, but I can't see a good reason not to. This way, the same options are accepted for the common cases (zero or one uses of -gdwarf-N), and an understandable error is thrown in the more complex case of multiple uses or it.
> Seems sad. I guess that can be a refactor. (EmitIntValue -> EmitInt16)
Yes, but that would be a separate patch.
http://llvm-reviews.chandlerc.com/D2697
More information about the llvm-commits
mailing list