[llvm-commits] [llvm] r86303 - in /llvm/trunk: lib/Target/ARM/AsmPrinter/ARMAsmPrinter.cpp test/CodeGen/ARM/2009-11-07-SubRegAsmPrinting.ll

Evan Cheng evan.cheng at apple.com
Fri Nov 6 16:34:10 PST 2009


On Nov 6, 2009, at 4:30 PM, Jakob Stoklund Olesen wrote:

>>
>> =
>> =====================================================================
>> --- llvm/trunk/lib/Target/ARM/AsmPrinter/ARMAsmPrinter.cpp (original)
>> +++ llvm/trunk/lib/Target/ARM/AsmPrinter/ARMAsmPrinter.cpp Fri Nov
>> 6 17:45:15 2009
>> @@ -347,6 +347,9 @@
>>
>> &ARM::DPR_VFP2RegClass);
>>      O << getRegisterName(DReg) << '[' << (RegNum & 1) << ']';
>>    } else {
>> +      if (unsigned SubReg = MO.getSubReg())
>> +        Reg = TRI->getSubReg(Reg, SubReg);
>> +
>>      O << getRegisterName(Reg);
>>    }
>>    break;
>
> I thought that the SubReg field was only to be used with virtual
> registers?
>
> Is this perhaps a bug in an earlier stage?

Right. It should not have survived earlier passes.

Evan

>
> /jakob
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list