[Lldb-commits] [PATCH] D44472: Add and fix some tests for PPC64

Alexandre Yukio Yamashita via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Mar 15 13:08:09 PDT 2018


alexandreyy added a comment.

In https://reviews.llvm.org/D44472#1038656, @labath wrote:

> 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...


I executed the tests again and we don't really need both powerpc64le and ppc64le.
Some tests required "ppc64le" to be built. But they are working now without it.
I modified the disassemble test as you suggested.


https://reviews.llvm.org/D44472





More information about the lldb-commits mailing list