diff options
| author | Chad Rosier <mcrosier@apple.com> | 2012-08-31 17:24:10 +0000 |
|---|---|---|
| committer | Chad Rosier <mcrosier@apple.com> | 2012-08-31 17:24:10 +0000 |
| commit | 9d1fc3672ba98b8ecd1ab5bdf048573238799b92 (patch) | |
| tree | 6f9ce17387f5aa5c1c0590e5b99879feb6820fa1 /llvm/lib/Target/ARM/AsmParser | |
| parent | 219c9d0dd3ebcfbf3e8710b1c3149c90041ea24f (diff) | |
| download | bcm5719-llvm-9d1fc3672ba98b8ecd1ab5bdf048573238799b92.tar.gz bcm5719-llvm-9d1fc3672ba98b8ecd1ab5bdf048573238799b92.zip | |
Add a comment to explain what's really going on.
llvm-svn: 163005
Diffstat (limited to 'llvm/lib/Target/ARM/AsmParser')
| -rw-r--r-- | llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp b/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp index 1fde5a2e79d..3ee3db5b028 100644 --- a/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp +++ b/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp @@ -5347,6 +5347,12 @@ validateInstruction(MCInst &Inst, case ARM::tMUL: { // The second source operand must be the same register as the destination // operand. + // + // In this case, we must directly check the parsed operands because the + // cvtThumbMultiply() function is written in such a way that it guarantees + // this first statement is always true for the new Inst. Essentially, the + // destination is unconditionally copied into the second source operand + // without checking to see if it matches what we actually parsed. if (Operands.size() == 6 && (((ARMOperand*)Operands[3])->getReg() != ((ARMOperand*)Operands[5])->getReg()) && |
