[Lldb-commits] [PATCH] D98197: [lldb] Fix DWARF-5 DW_FORM_implicit_const (used by GCC)
Jan Kratochvil via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Mar 9 00:54:11 PST 2021
jankratochvil marked 2 inline comments as done.
jankratochvil added a comment.
In D98197#2613149 <https://reviews.llvm.org/D98197#2613149>, @labath wrote:
> Could you simplify the test case?
> I guess we should also delete `test/Shell/Breakpoint/implicit_const_form_support.test`, as it's not very useful (the numbers it checks come from the line table, not from the implicit constants).
I have not deleted it myself; I find non-racy simple tests to be always good as they may catch some completely unrelated regression in the future.
Comment at: lldb/test/Shell/SymbolFile/DWARF/dwarf5-implicit-const.s:3
+# UNSUPPORTED: system-darwin, system-windows
+# REQUIRES: x86
> I hope these are not necessary, since you don't run the code here.
-# UNSUPPORTED: system-darwin, system-windows
-# REQUIRES: x86
We will see after all the bots run again.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits