[llvm-commits] [llvm] r137891 - in /llvm/trunk: lib/Target/ARM/AsmParser/ARMAsmParser.cpp test/MC/ARM/basic-thumb-instructions.s

[Jim Grosbach]

On Aug 17, 2011, at 5:47 PM, Chris Lattner wrote:

> 
> On Aug 17, 2011, at 3:57 PM, Jim Grosbach wrote:
> 
>> Author: grosbach
>> Date: Wed Aug 17 17:57:40 2011
>> New Revision: 137891
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=137891&view=rev
>> Log:
>> Thumb assembly parsing and encoding for B.
>> 
>> Modified:
>>   llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
>>   llvm/trunk/test/MC/ARM/basic-thumb-instructions.s
>> 
>> Modified: llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp?rev=137891&r1=137890&r2=137891&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp (original)
>> +++ llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp Wed Aug 17 17:57:40 2011
>> @@ -3035,6 +3035,10 @@
>>    if (Inst.getOperand(3).getImm() < 8)
>>      Inst.setOpcode(ARM::tADDi3);
>>    break;
>> +  case ARM::tBcc:
>> +    // If the conditional is AL, we really want tB.
>> +    if (Inst.getOperand(1).getImm() == ARMCC::AL)
>> +      Inst.setOpcode(ARM::tB);
>>  }
>> }
> 
> Not necessary for correctness, but please add a break;

Eep. Sorry about that. Fixed in r137941.