diff options
author | Patrik Hagglund <patrik.h.hagglund@ericsson.com> | 2014-04-28 12:12:27 +0000 |
---|---|---|
committer | Patrik Hagglund <patrik.h.hagglund@ericsson.com> | 2014-04-28 12:12:27 +0000 |
commit | 319983810a0d4cf4d10127e9a3df52600ee7de0a (patch) | |
tree | c767dbff2d54313a3dbb3b98053ceff4cfeac4fb /llvm/lib/Target/X86/Disassembler/X86Disassembler.cpp | |
parent | 7b839f833d8c10f149da7fbfe97718505ffedf19 (diff) | |
download | bcm5719-llvm-319983810a0d4cf4d10127e9a3df52600ee7de0a.tar.gz bcm5719-llvm-319983810a0d4cf4d10127e9a3df52600ee7de0a.zip |
Fix gcc -Wsign-compare warning in X86DisassemblerTables.cpp.
X86_MAX_OPERANDS is changed to unsigned.
Also, add range-based for loops for affected loops. This in turn
needed an ArrayRef instead of a pointer-to-array in
InternalInstruction.
llvm-svn: 207413
Diffstat (limited to 'llvm/lib/Target/X86/Disassembler/X86Disassembler.cpp')
-rw-r--r-- | llvm/lib/Target/X86/Disassembler/X86Disassembler.cpp | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/llvm/lib/Target/X86/Disassembler/X86Disassembler.cpp b/llvm/lib/Target/X86/Disassembler/X86Disassembler.cpp index 5658cf8df54..c36672578b4 100644 --- a/llvm/lib/Target/X86/Disassembler/X86Disassembler.cpp +++ b/llvm/lib/Target/X86/Disassembler/X86Disassembler.cpp @@ -787,13 +787,11 @@ static bool translateInstruction(MCInst &mcInst, mcInst.setOpcode(X86::XACQUIRE_PREFIX); } - int index; - insn.numImmediatesTranslated = 0; - for (index = 0; index < X86_MAX_OPERANDS; ++index) { - if (insn.operands[index].encoding != ENCODING_NONE) { - if (translateOperand(mcInst, insn.operands[index], insn, Dis)) { + for (const auto &Op : insn.operands) { + if (Op.encoding != ENCODING_NONE) { + if (translateOperand(mcInst, Op, insn, Dis)) { return true; } } |