Browse Source

Build: Build Userland with -O2, Kernel with -Os

For some reason I don't yet understand, building the kernel with -O2
produces a way-too-large kernel on some people's systems.

Since there are some really nice performance benefits from -O2 in
userspace, let's do a compromise and build Userland with -O2 but
put Kernel back into the -Os box for now.
Andreas Kling 4 years ago
parent
commit
f27eb315fc
3 changed files with 5 additions and 1 deletions
  1. 1 1
      CMakeLists.txt
  2. 2 0
      Kernel/CMakeLists.txt
  3. 2 0
      Userland/CMakeLists.txt

+ 1 - 1
CMakeLists.txt

@@ -153,7 +153,7 @@ set(CMAKE_CXX_LINK_FLAGS "-Wl,--hash-style=gnu,-z,relro,-z,now")
 # This will need to be revisited when the Loader supports RPATH/RUN_PATH.
 set(CMAKE_SKIP_RPATH TRUE)
 
-add_compile_options(-O2 -g1 -fno-exceptions -fstack-protector-strong -Wno-address-of-packed-member -Wundef -Wcast-qual -Wwrite-strings -Wimplicit-fallthrough -Wno-nonnull-compare -Wno-deprecated-copy -Wno-expansion-to-defined)
+add_compile_options(-g1 -fno-exceptions -fstack-protector-strong -Wno-address-of-packed-member -Wundef -Wcast-qual -Wwrite-strings -Wimplicit-fallthrough -Wno-nonnull-compare -Wno-deprecated-copy -Wno-expansion-to-defined)
 add_compile_options(-ffile-prefix-map=${CMAKE_SOURCE_DIR}=.)
 
 if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU")

+ 2 - 0
Kernel/CMakeLists.txt

@@ -1,3 +1,5 @@
+add_compile_options(-Os)
+
 set(KERNEL_HEAP_SOURCES
     Heap/SlabAllocator.cpp
     Heap/kmalloc.cpp

+ 2 - 0
Userland/CMakeLists.txt

@@ -1,3 +1,5 @@
+add_compile_options(-O2)
+
 add_subdirectory(Applications)
 add_subdirectory(Demos)
 add_subdirectory(DevTools)