[Lldb-commits] [PATCH] D127741: [Disassembler] Add +all CPU feature to subtargetInfo and use it for disassembling AArch64
Denis via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Jun 15 00:27:39 PDT 2022
treapster added a comment.
In D127741#3583559 <https://reviews.llvm.org/D127741#3583559>, @DavidSpickett wrote:
>
> I agree that objdump should have some kind of "max" setting, even a default. However there will still need to be a way to test what I referred to.
As you said, ` instruction requires bla` is part of assembler, not disassembler, so it is not affected by the patch. The problem here is that tests do `CHECK-UNKNOWN: 83 a9 91 c0 <unknown>` which basically checks that objdump cannot disassemble instruction that was assembled in the very same test. If we change that, we can make `+all` default attribute for objdump and it will disassemble everything the same way GNU objdump and lldb do.
> What might be possible is to write the testing such that it doesn't use llvm-objdump. I haven't seen the structure of the tests perhaps this would mean duplicating tests all over the place.
Why not use objdump? If we only check that instructions get disassembled, it will work fine.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127741/new/
https://reviews.llvm.org/D127741
More information about the lldb-commits
mailing list