[PATCH] ARM: Add support for ARM modified immediate syntax - Take	3
    Asiri Rathnayake 
    asiri.rathnayake at arm.com
       
    Mon Dec  1 07:48:40 PST 2014
    
    
  
>>! In D6408#12, @rmaprath wrote:
> Hi Tim,
> 
>> ================
>> Comment at: lib/Target/ARM/AsmParser/ARMAsmParser.cpp:4463-4465
>> @@ +4462,5 @@
>> +
>> +  if (Parser.getTok().isNot(AsmToken::Comma)) {
>> +    // Consider [mov r0, #-10], which is aliased with mvn. We cannot fail
>> +    // the parse here.
>> +    Operands.push_back(ARMOperand::CreateImm(Imm1Exp, Sx1, Ex1));
>> ----------------
>> I don't understand this one. Wouldn't that example have been covered by AsmToken::EndOfStatement above?
> 
> -10 (0xfffffff6 in two's complement) is not representable as a modified immediate. 
I.e (Enc == -1).
--Asiri
http://reviews.llvm.org/D6408
    
    
More information about the llvm-commits
mailing list