enable optimization

Signed-off-by: Jo5ta <[email protected]>

Author: Jo5ta

CMakePresets.json

@@ -13,6 +13,9 @@
             "cacheVariables": {
                 "CMAKE_EXPORT_COMPILE_COMMANDS": "ON",
                 "CMAKE_BUILD_TYPE": "Debug",
+                "CMAKE_INTERPROCEDURAL_OPTIMIZATION": "ON",
+                "CMAKE_CXX_FLAGS_DEBUG": "-Og",
+                "CMAKE_C_FLAGS_DEBUG": "-Og",
                 "STANDALONE_PROJECT": "ON"
             }
         },
@@ -23,19 +26,6 @@
                 "CLLTK_EXAMPLES": "ON"
             }
         },
-        {
-            "name": "release",
-            "inherits": "default",
-            "displayName": "Release",
-            "description": "Optimized release build with LTO.",
-            "cacheVariables": {
-                "CMAKE_BUILD_TYPE": "Release",
-                "CMAKE_CXX_FLAGS_RELEASE": "-O3 -flto -DNDEBUG",
-                "CMAKE_C_FLAGS_RELEASE": "-O3 -flto -DNDEBUG",
-                "CMAKE_INTERPROCEDURAL_OPTIMIZATION": "TRUE",
-                "CLLTK_TESTS": "OFF"
-            }
-        },
         {
             "name": "rpm",
             "inherits": "default",
@@ -71,13 +61,6 @@
             "targets": [
                 "all"
             ]
-        },
-        {
-            "name": "release",
-            "configurePreset": "release",
-            "targets": [
-                "all"
-            ]
         }
     ],
     "testPresets": [

VERSION.md

@@ -10,6 +10,7 @@ refactor: reorganize include structure and improve project layout
 - Restructured decoder_tool and snapshot_library to use consistent directory layout and public headers.
 - Fixed include paths in all examples, tests, and kernel module code to reflect new structure.
 - Removed redundant CMake logic and improved target properties.
+- Enable optimization
 ## 1.2.47
 - fix macros
 - add more cmake options

command_line_tool/commands/decode/decoder.cpp

@@ -128,11 +128,9 @@ static void add_decode_command(CLI::App &app)
 	});
 }
 
-static int init_function() noexcept
+static void init_function() noexcept
 {
 	auto [app, lock] = CommonLowLevelTracingKit::cmd::interface::acquireMainApp();
 	add_decode_command(app);
-	return 0;
 }
-
-static const int dummy = init_function();
+COMMAND_INIT(init_function);

command_line_tool/commands/snapshot/snapshot.cpp

@@ -65,11 +65,10 @@ static void add_snapshot_command(CLI::App &app)
 		[&]() { take_snapshot(output_file_name, tracepoints, compress, bucket_size, verbose); });
 }
 
-static int init_function() noexcept
+
+static void init_function() noexcept
 {
 	auto [app, lock] = CommonLowLevelTracingKit::cmd::interface::acquireMainApp();
 	add_snapshot_command(app);
-	return 0;
 }
-
-static const int dummy = init_function();
+COMMAND_INIT(init_function);

command_line_tool/commands/trace/tracebuffer.cpp

@@ -41,11 +41,10 @@ static void add_create_tracebuffer_command(CLI::App &app)
 	command->callback([]() { clltk_dynamic_tracebuffer_creation(tracebuffer.c_str(), size); });
 }
 
-static int init_function() noexcept
+static void init_function() noexcept
 {
 	auto [app, lock] = CommonLowLevelTracingKit::cmd::interface::acquireMainApp();
 	add_create_tracebuffer_command(app);
-	return 0;
 }
+COMMAND_INIT(init_function);
 
-static const int dummy = init_function();

command_line_tool/commands/trace/tracepipe.cpp

@@ -122,11 +122,9 @@ class Command
 	}
 };
 
-static int init_function() noexcept
+static void init_function() noexcept
 {
 	auto [app, lock] = CommonLowLevelTracingKit::cmd::interface::acquireMainApp();
 	Command::AddCommand(app);
-	return 0;
 }
-
-static const int dummy = init_function();
+COMMAND_INIT(init_function);

command_line_tool/commands/trace/tracepoint.cpp

@@ -69,11 +69,9 @@ static void add_create_tracepoint_command(CLI::App &app)
 	});
 }
 
-static int init_function() noexcept
+static void init_function() noexcept
 {
 	auto [app, lock] = CommonLowLevelTracingKit::cmd::interface::acquireMainApp();
 	add_create_tracepoint_command(app);
-	return 0;
 }
-
-static const int dummy = init_function();
+COMMAND_INIT(init_function);

command_line_tool/interface/commands/interface.hpp

@@ -8,11 +8,20 @@
 #include "CLI/Config.hpp"	  // IWYU pragma: export
 #include "CLI/Formatter.hpp"  // IWYU pragma: export
 #include "CLI/Validators.hpp" // IWYU pragma: export
+#if __has_include("CLI/ExtraValidators.hpp")
+#include "CLI/ExtraValidators.hpp" // IWYU pragma: export
+#endif
 #include <CLI/Error.hpp>	  // IWYU pragma: export
 #include <CLI/Option.hpp>	  // IWYU pragma: export
 
 #include <mutex>
 
+typedef void (*init_fn)(void);
+#define COMMAND_INIT(func) \
+    __attribute__((retain, used, section("clltk_cmdinit"))) \
+    static init_fn _init_##func##_ptr = func
+
+
 namespace CommonLowLevelTracingKit::cmd::interface
 {
 using MainAppHandle = std::pair<CLI::App &, std::unique_lock<std::mutex>>;

command_line_tool/main/main.cpp

@@ -61,8 +61,17 @@ CommonLowLevelTracingKit::cmd::interface::acquireMainApp(void)
 	return {*mainApp, std::unique_lock{mainAppLock}};
 }
 
+void call_all_init_functions(void) {
+	extern init_fn __start_clltk_cmdinit;
+	extern init_fn __stop_clltk_cmdinit;
+    for (init_fn *p = &__start_clltk_cmdinit; p < &__stop_clltk_cmdinit; p++) {
+        (*p)();
+    }
+}
+
 int main(int argc, const char **argv)
 {
+	call_all_init_functions();
 	auto [app, lock] = CommonLowLevelTracingKit::cmd::interface::acquireMainApp();
 	if (argc == 1) {
 		std::cout << app.help() << std::endl;