-
Show log
Commit
-
Hash : cf888486
Author :
Date : 2016-02-02T23:17:06
Use consistent formatting in ARM NEON SIMD code There aren't really any best practices to follow here. I tried as best as I could to adopt a standard that would ease any future maintenance burdens. The basic tenets of that standard are: * Assembly instructions always start on Column 5, and operands always start on Column 21, except: - The instruction and operand can be indented (usually by 2 spaces) to indicate a separate instruction stream. - If the instruction is within an enclosing .if block in a macro, it should always be indented relative to the .if block. * Comments are placed with an eye toward readability. There are always at least 2 spaces between the end of a line of code and the associated in-line comment. Where it made sense, I tried to line up the comments in blocks, and some were shifted right to avoid overlap with neighboring instruction lines. Not an exact science. * Assembler directives and macros use 2-space indenting rules. .if blocks are indented relative to the macro, and code within the .if blocks is indented relative to the .if directive. * No extraneous spaces between operands. Lining up the operands vertically did not really improve readability-- personally, I think it made it worse, since my eye would tend to lose its place in the uniform columns of characters. Also, code with a lot of vertical alignment is really hard to maintain, since changing one line could necessitate changing a bunch of other lines to avoid spoiling the alignment. * No extraneous spaces in #defines or other directives. In general, the only extraneous spaces (other than indenting spaces) are between: - Instructions and operands - Operands and in-line comments This standard should be more or less in keeping with other formatting standards used within the project.
-
Files
- CMakeLists.txt
- Makefile.am
- jccolext-altivec.c
- jccolext-mmx.asm
- jccolext-sse2-64.asm
- jccolext-sse2.asm
- jccolor-altivec.c
- jccolor-mmx.asm
- jccolor-sse2-64.asm
- jccolor-sse2.asm
- jcgray-altivec.c
- jcgray-mmx.asm
- jcgray-sse2-64.asm
- jcgray-sse2.asm
- jcgryext-altivec.c
- jcgryext-mmx.asm
- jcgryext-sse2-64.asm
- jcgryext-sse2.asm
- jchuff-sse2-64.asm
- jchuff-sse2.asm
- jcolsamp.inc
- jcsample-altivec.c
- jcsample-mmx.asm
- jcsample-sse2-64.asm
- jcsample-sse2.asm
- jcsample.h
- jdcolext-altivec.c
- jdcolext-mmx.asm
- jdcolext-sse2-64.asm
- jdcolext-sse2.asm
- jdcolor-altivec.c
- jdcolor-mmx.asm
- jdcolor-sse2-64.asm
- jdcolor-sse2.asm
- jdct.inc
- jdmerge-altivec.c
- jdmerge-mmx.asm
- jdmerge-sse2-64.asm
- jdmerge-sse2.asm
- jdmrgext-altivec.c
- jdmrgext-mmx.asm
- jdmrgext-sse2-64.asm
- jdmrgext-sse2.asm
- jdsample-altivec.c
- jdsample-mmx.asm
- jdsample-sse2-64.asm
- jdsample-sse2.asm
- jfdctflt-3dn.asm
- jfdctflt-sse-64.asm
- jfdctflt-sse.asm
- jfdctfst-altivec.c
- jfdctfst-mmx.asm
- jfdctfst-sse2-64.asm
- jfdctfst-sse2.asm
- jfdctint-altivec.c
- jfdctint-mmx.asm
- jfdctint-sse2-64.asm
- jfdctint-sse2.asm
- jidctflt-3dn.asm
- jidctflt-sse.asm
- jidctflt-sse2-64.asm
- jidctflt-sse2.asm
- jidctfst-altivec.c
- jidctfst-mmx.asm
- jidctfst-sse2-64.asm
- jidctfst-sse2.asm
- jidctint-altivec.c
- jidctint-mmx.asm
- jidctint-sse2-64.asm
- jidctint-sse2.asm
- jidctred-mmx.asm
- jidctred-sse2-64.asm
- jidctred-sse2.asm
- jpeg_nbits_table.inc
- jquant-3dn.asm
- jquant-mmx.asm
- jquant-sse.asm
- jquantf-sse2-64.asm
- jquantf-sse2.asm
- jquanti-altivec.c
- jquanti-sse2-64.asm
- jquanti-sse2.asm
- jsimd.h
- jsimd_altivec.h
- jsimd_arm.c
- jsimd_arm64.c
- jsimd_arm64_neon.S
- jsimd_arm_neon.S
- jsimd_i386.c
- jsimd_mips.c
- jsimd_mips_dspr2.S
- jsimd_mips_dspr2_asm.h
- jsimd_powerpc.c
- jsimd_x86_64.c
- jsimdcfg.inc.h
- jsimdcpu.asm
- jsimdext.inc
- nasm_lt.sh
