[Lldb-commits] [PATCH] D22294: Add functionality for rewriting symbols
Francis Ricci via lldb-commits
lldb-commits at lists.llvm.org
Wed Jul 13 09:27:13 PDT 2016
fjricci marked 3 inline comments as done.
Comment at: packages/Python/lldbsuite/test/lang/c/symbol_rewriter/TestSymbolRewriter.py:28
@@ +27,3 @@
+ # Clang does not rewrite dwarf debug info, so it must be stripped
+ subprocess.check_call(['strip', '-g', exe])
> This is not going to work for remote targets on different architectures. If you don't need debug info, could you just avoid generating it in the first place (-g0 ?). Maybe then you would be able to get this working on windows as well.
> If it doesn't work, then we should wire up this call to go through the Makefile, as it already knows how to find the right toolchain for cross-compilation.
Unfortunately, the '-g0' override will only work for dwarf tests. However, based on TestWithLimitDebugInfo, it looks like we can skip the test for non-dwarf symbols (`@skipIf(debug_info=no_match(["dwarf"]))`). I think this is probably the cleanest way to go.
More information about the lldb-commits