Fix the Align() utility which tries to align the raw disassembly with the edis'ed disassembly
so that both the opcode and the operands are aligned with the rest of output.
Comment out the code related to force_raw mode when disassembling arm or thumb for now.
It testing goes ok, we will remove the section of code related to force_raw.
git-svn-id: https://llvm.org/svn/llvm-project/llvdb/trunk@131910 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/source/Plugins/Disassembler/llvm/DisassemblerLLVM.cpp b/source/Plugins/Disassembler/llvm/DisassemblerLLVM.cpp
index ea31a82..4b5d1dc 100644
--- a/source/Plugins/Disassembler/llvm/DisassemblerLLVM.cpp
+++ b/source/Plugins/Disassembler/llvm/DisassemblerLLVM.cpp
@@ -109,11 +109,15 @@
Str = Str.substr(0, Str.size()-1);
}
static void
-Align(Stream *s, const char *str)
+Align(Stream *s, const char *str, size_t opcodeColWidth, size_t operandColWidth)
{
llvm::StringRef raw_disasm(str);
StripSpaces(raw_disasm);
- s->PutCString(raw_disasm.str().c_str());
+ // Split the raw disassembly into opcode and operands.
+ std::pair<llvm::StringRef, llvm::StringRef> p = raw_disasm.split('\t');
+ PadString(s, p.first, opcodeColWidth);
+ if (!p.second.empty())
+ PadString(s, p.second, operandColWidth);
}
void
@@ -173,10 +177,12 @@
// FIXME!!!
/* Remove the following section of code related to force_raw .... */
+ /*
bool force_raw = m_arch_type == llvm::Triple::arm ||
m_arch_type == llvm::Triple::thumb;
if (!raw)
raw = force_raw;
+ */
/* .... when we fix the edis for arm/thumb. */
if (!raw)
@@ -349,7 +355,7 @@
if (EDGetInstString(&str, m_inst))
return;
- Align(s, str);
+ Align(s, str, opcodeColumnWidth, operandColumnWidth);
}
else
{
@@ -380,7 +386,7 @@
{
// EDis fails to parse the tokens of this inst. Need to align this
// raw disassembly's opcode with the rest of output.
- Align(s, str);
+ Align(s, str, opcodeColumnWidth, operandColumnWidth);
}
}
}