[llvm-commits] [llvm] r161768 - in /llvm/trunk: lib/Target/ARM/ARMAsmPrinter.cpp test/CodeGen/ARM/arm-modifier.ll

[Eric Christopher]

On Aug 14, 2012, at 4:17 PM, Bill Wendling <wendling at apple.com> wrote:

> On Aug 13, 2012, at 11:18 AM, Eric Christopher <echristo at apple.com> wrote:
> 
>> Author: echristo
>> Date: Mon Aug 13 13:18:52 2012
>> New Revision: 161768
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=161768&view=rev
>> Log:
>> Add support for the %H output modifier.
>> 
>> Patch by Weiming Zhao.
>> 
>> Modified:
>>   llvm/trunk/lib/Target/ARM/ARMAsmPrinter.cpp
>>   llvm/trunk/test/CodeGen/ARM/arm-modifier.ll
>> 
>> Modified: llvm/trunk/lib/Target/ARM/ARMAsmPrinter.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMAsmPrinter.cpp?rev=161768&r1=161767&r2=161768&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/ARM/ARMAsmPrinter.cpp (original)
>> +++ llvm/trunk/lib/Target/ARM/ARMAsmPrinter.cpp Mon Aug 13 13:18:52 2012
>> @@ -529,10 +529,23 @@
>>      return false;
>>    }
>> 
>> -    // These modifiers are not yet supported.
>> +    // This modifier is not yet supported.
>>    case 'h': // A range of VFP/NEON registers suitable for VLD1/VST1.
>> -    case 'H': // The highest-numbered register of a pair.
>>      return true;
>> +    case 'H': // The highest-numbered register of a pair.
> 
> Don't you need a '{' right here?

At least I should yeah. Thanks:

	M	lib/Target/ARM/ARMAsmPrinter.cpp
Committed r161906


-eric