Commit 2008d89f authored by WANG Xuerui's avatar WANG Xuerui Committed by Song Liu

raid6: test: cosmetic cleanups for the test Makefile

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: default avatarWANG Xuerui <git@xen0n.name>
Link: https://lore.kernel.org/r/20230731104911.411964-4-kernel@xen0n.nameSigned-off-by: default avatarSong Liu <song@kernel.org>
parent 9dd6e1da
...@@ -6,14 +6,15 @@ ...@@ -6,14 +6,15 @@
pound := \# pound := \#
CC = gcc # Adjust as desired
OPTFLAGS = -O2 # Adjust as desired CC = gcc
CFLAGS = -I.. -I ../../../include -g $(OPTFLAGS) OPTFLAGS = -O2
LD = ld CFLAGS = -I.. -I ../../../include -g $(OPTFLAGS)
AWK = awk -f LD = ld
AR = ar AWK = awk -f
RANLIB = ranlib AR = ar
OBJS = int1.o int2.o int4.o int8.o int16.o int32.o recov.o algos.o tables.o 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/) ARCH := $(shell uname -m 2>/dev/null | sed -e /s/i.86/i386/)
ifeq ($(ARCH),i386) ifeq ($(ARCH),i386)
...@@ -37,9 +38,9 @@ endif ...@@ -37,9 +38,9 @@ endif
ifeq ($(IS_X86),yes) ifeq ($(IS_X86),yes)
OBJS += mmx.o sse1.o sse2.o avx2.o recov_ssse3.o recov_avx2.o avx512.o recov_avx512.o OBJS += mmx.o sse1.o sse2.o avx2.o recov_ssse3.o recov_avx2.o avx512.o recov_avx512.o
CFLAGS += -DCONFIG_X86 CFLAGS += -DCONFIG_X86
CFLAGS += $(shell echo "vpmovm2b %k1, %zmm5" | \ CFLAGS += $(shell echo "vpmovm2b %k1, %zmm5" | \
gcc -c -x assembler - >/dev/null 2>&1 && \ gcc -c -x assembler - >/dev/null 2>&1 && \
rm ./-.o && echo -DCONFIG_AS_AVX512=1) rm ./-.o && echo -DCONFIG_AS_AVX512=1)
else ifeq ($(HAS_NEON),yes) else ifeq ($(HAS_NEON),yes)
OBJS += neon.o neon1.o neon2.o neon4.o neon8.o recov_neon.o recov_neon_inner.o OBJS += neon.o neon1.o neon2.o neon4.o neon8.o recov_neon.o recov_neon_inner.o
CFLAGS += -DCONFIG_KERNEL_MODE_NEON=1 CFLAGS += -DCONFIG_KERNEL_MODE_NEON=1
...@@ -63,12 +64,12 @@ endif ...@@ -63,12 +64,12 @@ endif
%.uc: ../%.uc %.uc: ../%.uc
cp -f $< $@ cp -f $< $@
all: raid6.a raid6test all: raid6.a raid6test
raid6.a: $(OBJS) raid6.a: $(OBJS)
rm -f $@ rm -f $@
$(AR) cq $@ $^ $(AR) cq $@ $^
$(RANLIB) $@ $(RANLIB) $@
raid6test: test.c raid6.a raid6test: test.c raid6.a
$(CC) $(CFLAGS) -o raid6test $^ $(CC) $(CFLAGS) -o raid6test $^
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment