[llvm-commits] [llvm] r154839 - /llvm/trunk/test/MC/ARM/neon-shift-encoding.s

[Jim Grosbach]

No hugely strong reason. The main one is that it's already pretty tight keeping things inside 80 columns.

I'll play with some formatting and see what things would look like with regards to that and general readability if we updated them all. If it's a net gain in readability, it should be easy enough to write a script to update the files to a consistent format.

-Jim

On Apr 16, 2012, at 11:57 AM, Chris Lattner <clattner at apple.com> wrote:

> Nice!
> 
> Out of curiosity, why not put the check lines on the same line as the asm instruction?  E.g.:
> 
> vshll.s8	q8, d16, #7 @ CHECK: vshll.s8	q8, d16, #7  @ encoding: [0x30,0x0a,0xcf,0xf2]
> 
> -Chris
> 
> On Apr 16, 2012, at 11:03 AM, Jim Grosbach wrote:
> 
>> Author: grosbach
>> Date: Mon Apr 16 13:03:14 2012
>> New Revision: 154839
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=154839&view=rev
>> Log:
>> Tidy up. Test formatting.
>