diff options
author | Zachary Turner <zturner@google.com> | 2016-11-17 18:08:12 +0000 |
---|---|---|
committer | Zachary Turner <zturner@google.com> | 2016-11-17 18:08:12 +0000 |
commit | 31d97a5c8ab78c619deada0cdb1fcf64021d25dd (patch) | |
tree | a1f642a22b8975a80013c7ab8bc98a7ea763346e /lldb/source/Core/Disassembler.cpp | |
parent | ff0382c16190cd2b1bc47b27c5db0570b90f1d39 (diff) | |
download | bcm5719-llvm-31d97a5c8ab78c619deada0cdb1fcf64021d25dd.tar.gz bcm5719-llvm-31d97a5c8ab78c619deada0cdb1fcf64021d25dd.zip |
Rewrite all Property related functions in terms of StringRef.
This was a bit tricky, especially for things like
OptionValueArray and OptionValueDictionary since they do some
funky string parsing. Rather than try to re-write line-by-line
I tried to make the StringRef usage idiomatic, even though
it meant often re-writing from scratch large blocks of code
in a different way while keeping true to the original intent.
The finished code is a big improvement though, and often much
shorter than the original code. All tests and unit tests
pass on Windows and Linux.
llvm-svn: 287242
Diffstat (limited to 'lldb/source/Core/Disassembler.cpp')
-rw-r--r-- | lldb/source/Core/Disassembler.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lldb/source/Core/Disassembler.cpp b/lldb/source/Core/Disassembler.cpp index c55ad1c1755..7cac29491c8 100644 --- a/lldb/source/Core/Disassembler.cpp +++ b/lldb/source/Core/Disassembler.cpp @@ -1321,9 +1321,8 @@ void PseudoInstruction::SetOpcode(size_t opcode_size, void *opcode_data) { } } -void PseudoInstruction::SetDescription(const char *description) { - if (description && strlen(description) > 0) - m_description = description; +void PseudoInstruction::SetDescription(llvm::StringRef description) { + m_description = description; } Instruction::Operand Instruction::Operand::BuildRegister(ConstString &r) { |