build: Remove CPU feature config options

This change removes the Kconfig options selecting CPU feature set (i.e.,
`-march=` compiler flag) from the configuration of this library.
CPU feature flags are set elsewhere in the build configuration and the
intrinsics library can already seamlessly pick up these features via
preprocessor defines.

Signed-off-by: Andrei Tatar <andrei@unikraft.io>
Reviewed-by: Maria Sfiraiala <maria.sfiraiala@gmail.com>
Reviewed-by: Radu Nichita <radunichita99@gmail.com>
Approved-by: Razvan Deaconescu <razvand@unikraft.io>
Tested-by: Unikraft CI <monkey@unikraft.io>
GitHub-Closes: #3

diff --git a/Config.uk b/Config.uk
index 413b137ea29cef53ba917853c9ce5852df5f8617..7afb70a1bf2863bdeb49acc2191b131ffc2bb41a 100644 (file)
--- a/Config.uk
+++ b/Config.uk
@@ -1,25 +1,3 @@
-menuconfig LIBINTEL_INTRINSICS
-    bool "Intel Intrinsics - C style functions that provide access Intel instructions"
+config LIBINTEL_INTRINSICS
+    bool "Intel Intrinsics - C style functions that provide access to x86 instructions"
     default n
-
-if LIBINTEL_INTRINSICS
-config SIMD_SSE
-    bool "Enable SSE support"
-    default n
-
-config SIMD_SSE2
-    bool "Enable SSE2 support"
-    default n
-
-config SIMD_SSE3
-    bool "Enable SSE3 support"
-    default n
-
-config SIMD_SSE4
-    bool "Enable SSE4 support"
-    default n
-
-config SIMD_AVX
-    bool "Enable AVX support"
-    default n
-endif

diff --git a/Makefile.uk b/Makefile.uk
index 722ea13f4dea39c329e5d2deb44c842bce81d7c9..0131610d45422e389229adeb892b20dd2bd3b26c 100644 (file)
--- a/Makefile.uk
+++ b/Makefile.uk
@@ -1,8 +1,10 @@
 #  libintel_intrinsics Makefile.uk
 #
 #  Authors: Vlad-Andrei Badoiu <vlad_andrei.badoiu@stud.acs.upb.ro>
+#           Andrei Tatar <andrei@unikraft.io>
 #
 #  Copyright (c) 2019, University Politehnica of Bucharest. All rights reserved.
+#  Copyright (c) 2023, Unikraft GmbH and The Unikraft Authors.
 #
 #  Redistribution and use in source and binary forms, with or without
 #  modification, are permitted provided that the following conditions
@@ -38,30 +40,7 @@ $(eval $(call addlib_s,libintel_intrinsics,$(CONFIG_LIBINTEL_INTRINSICS)))
 ################################################################################
 # Library includes
 ################################################################################
+ifeq ($(CONFIG_LIBINTEL_INTRINSICS),y)
 CINCLUDES-$(CONFIG_LIBINTEL_INTRINSICS) += -I$(LIBINTEL_INTRINSICS_BASE)/include
 CXXINCLUDES-$(CONFIG_LIBINTEL_INTRINSICS) += -I$(LIBINTEL_INTRINSICS_BASE)/include
-
-ifdef CONFIG_SIMD_SSE
-CFLAGS         += -msse -mfpmath=sse
-CXXFLAGS       += -msse -mfpmath=sse
-endif
-
-ifdef CONFIG_SIMD_SSE2
-CFLAGS         += -msse2
-CXXFLAGS       += -msse2
-endif
-
-ifdef CONFIG_SIMD_SSE3
-CFLAGS         += -msse3
-CXXFLAGS       += -msse3
-endif
-
-ifdef CONFIG_SIMD_SSE4
-CFLAGS         += -msse4
-CXXFLAGS       += -msse4
-endif
-
-ifdef CONFIG_SIMD_AVX
-CFLAGS         += -mavx
-CXXFLAGS       += -mavx
 endif