fix: add -O2 flag and sha optimizations for arm

spiral-ladder · spiral-ladder · commit d70db1cec8a1 · 2025-10-20T15:47:44.000+08:00
This missing c flag as well as the macro were causing some regressions in performance within lodestar-bun versus production blst-ts in ChainSafe/lodestar-bun#18.
diff --git a/build.zig b/build.zig
@@ -11,6 +11,7 @@ pub fn build(b: *std.Build) !void {
     var c_flags = std.ArrayList([]const u8).init(b.allocator);
     defer c_flags.deinit();
 
+    try c_flags.append("-O2");
     try c_flags.append("-fno-builtin");
     try c_flags.append("-Wno-unused-function");
     try c_flags.append("-Wno-unused-command-line-argument");
@@ -38,6 +39,8 @@ pub fn build(b: *std.Build) !void {
         }
     }
 
+    if (target.result.cpu.arch == .aarch64) lib.root_module.addCMacro("__ARM_FEATURE_CRYPTO", "1");
+
     if (target.result.cpu.arch != .x86_64 and
         target.result.cpu.arch != .aarch64)
     {

Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,7 @@ pub fn build(b: *std.Build) !void {`
`11`	`11`	`var c_flags = std.ArrayList([]const u8).init(b.allocator);`
`12`	`12`	`defer c_flags.deinit();`
`13`	`13`
	`14`	`+ try c_flags.append("-O2");`
`14`	`15`	`try c_flags.append("-fno-builtin");`
`15`	`16`	`try c_flags.append("-Wno-unused-function");`
`16`	`17`	`try c_flags.append("-Wno-unused-command-line-argument");`
`@@ -38,6 +39,8 @@ pub fn build(b: *std.Build) !void {`
`38`	`39`	`}`
`39`	`40`	`}`
`40`	`41`
	`42`	`+ if (target.result.cpu.arch == .aarch64) lib.root_module.addCMacro("__ARM_FEATURE_CRYPTO", "1");`
	`43`	`+`
`41`	`44`	`if (target.result.cpu.arch != .x86_64 and`
`42`	`45`	`target.result.cpu.arch != .aarch64)`
`43`	`46`	`{`