[Lldb-commits] [PATCH] D44472: Add and fix some tests for PPC64
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Mar 15 06:39:02 PDT 2018
labath added a comment.
I wonder if we shouldn't just fix the TestDisassembleBreakpoint to not require adding every single architecture. That test was added because lldb-server was not removing the traps from the memory read packets. This is something that is completely independent of us later trying to disassemble the memory we got back. To test that, it should be sufficient to compare the memory contents before and after setting a breakpoint.
Guessing what instructions will the compiler emit for a particular piece of C will just make the test fragile...
Comment at: packages/Python/lldbsuite/test/make/Makefile.rules:189
+ ifeq "$(ARCH)" "ppc64le"
+ override ARCH := $(subst ppc64le,64,$(ARCH))
Why do you need both powerpc64le and ppc64le?
More information about the lldb-commits