raid6: test: cosmetic cleanups for the test Makefile
authorWANG Xuerui <git@xen0n.name>
Mon, 31 Jul 2023 10:49:09 +0000 (18:49 +0800)
committerSong Liu <song@kernel.org>
Tue, 15 Aug 2023 16:40:27 +0000 (09:40 -0700)
Use tabs/spaces consistently: hard tabs for marking recipe lines only,
spaces for everything else.

Also, the OPTFLAGS declaration actually included the tabs preceding the
line comment, making compiler invocation lines unnecessarily long. As
the entire block of declarations are meant for ad-hoc customization
(otherwise they would probably make use of `?=` instead of `=`), move
the "Adjust as desired" comment above the block too to fix the long
invocation lines.

Signed-off-by: WANG Xuerui <git@xen0n.name>
Link: https://lore.kernel.org/r/20230731104911.411964-4-kernel@xen0n.name
Signed-off-by: Song Liu <song@kernel.org>
lib/raid6/test/Makefile

index 4fb7700a741bdf289eda14ff0d2a6d27c9b8ea10..143cda60faa12c70ddd70796d0f64c9af77ae3e8 100644 (file)
@@ -6,14 +6,15 @@
 
 pound := \#
 
-CC      = gcc
-OPTFLAGS = -O2                 # Adjust as desired
-CFLAGS  = -I.. -I ../../../include -g $(OPTFLAGS)
-LD      = ld
-AWK     = awk -f
-AR      = ar
-RANLIB  = ranlib
-OBJS    = int1.o int2.o int4.o int8.o int16.o int32.o recov.o algos.o tables.o
+# Adjust as desired
+CC       = gcc
+OPTFLAGS = -O2
+CFLAGS   = -I.. -I ../../../include -g $(OPTFLAGS)
+LD       = ld
+AWK      = awk -f
+AR       = ar
+RANLIB   = ranlib
+OBJS     = int1.o int2.o int4.o int8.o int16.o int32.o recov.o algos.o tables.o
 
 ARCH := $(shell uname -m 2>/dev/null | sed -e /s/i.86/i386/)
 ifeq ($(ARCH),i386)
@@ -37,9 +38,9 @@ endif
 ifeq ($(IS_X86),yes)
         OBJS   += mmx.o sse1.o sse2.o avx2.o recov_ssse3.o recov_avx2.o avx512.o recov_avx512.o
         CFLAGS += -DCONFIG_X86
-       CFLAGS += $(shell echo "vpmovm2b %k1, %zmm5" |          \
-                   gcc -c -x assembler - >/dev/null 2>&1 &&    \
-                   rm ./-.o && echo -DCONFIG_AS_AVX512=1)
+        CFLAGS += $(shell echo "vpmovm2b %k1, %zmm5" |          \
+                    gcc -c -x assembler - >/dev/null 2>&1 &&    \
+                    rm ./-.o && echo -DCONFIG_AS_AVX512=1)
 else ifeq ($(HAS_NEON),yes)
         OBJS   += neon.o neon1.o neon2.o neon4.o neon8.o recov_neon.o recov_neon_inner.o
         CFLAGS += -DCONFIG_KERNEL_MODE_NEON=1
@@ -63,12 +64,12 @@ endif
 %.uc: ../%.uc
        cp -f $< $@
 
-all:   raid6.a raid6test
+all: raid6.a raid6test
 
 raid6.a: $(OBJS)
-        rm -f $@
-        $(AR) cq $@ $^
-        $(RANLIB) $@
+       rm -f $@
+       $(AR) cq $@ $^
+       $(RANLIB) $@
 
 raid6test: test.c raid6.a
        $(CC) $(CFLAGS) -o raid6test $^