going back to using -O1 instead of -O0 when building profiling image

Author: marc-casavant

src/tests/multi-server/scripts/docker/build/Dockerfile.multi-server-prof

@@ -16,27 +16,20 @@ RUN autoreconf
 
 # CFLAGS used for profiling build:
 #   -g3                          Maximum debug info for callgrind symbol resolution
-#   -O0                          No optimisation: preserves the exact call structure of the source.
-#                                Disables dead-code elimination, tail-call optimisation, and all
-#                                other transforms that remove or reorder calls.
-#   -fno-omit-frame-pointer      Keep frame pointers so callgrind can walk the call stack
-#   -fno-inline                  Disable inlining (including CC_HINT(flatten)) so call edges appear in the graph
-#   -Dalways_inline=             Strip __attribute__((always_inline)) which -fno-inline does not suppress
-#   -fno-plt                     Eliminate PLT trampolines on cross-library calls (e.g. worker.c ->
-#                                interpret.c). Without this, the linker inserts a PLT stub with no
-#                                DWARF debug info, which appears as an unresolved "???" node in the
-#                                callgrind call graph. With -fno-plt, the compiler generates a direct
-#                                GOT-based indirect call that callgrind resolves correctly.
-#   -fno-builtin                 Prevent replacement of standard library calls (memcpy, strlen, etc.)
-#                                with inline intrinsics, which would erase them from the call graph.
-#   -fno-optimize-sibling-calls  Prevent tail-call elimination converting call+ret into a jump,
-#                                which removes the call edge from the call graph. Already implied
-#                                by -O0 but stated explicitly as a safeguard.
+#   -O1                          Basic optimisation for realistic hotspot costs; inlining/vectorisation/
+#                                unrolling disabled below so the call graph matches the source.
+#   -fno-omit-frame-pointer      Keep frame pointers for callgrind stack walking
+#   -fno-inline                  Preserve call edges (suppresses CC_HINT(flatten) too)
+#   -Dalways_inline=             Strip always_inline, which -fno-inline does not suppress
+#   -fno-plt                     Resolve cross-library calls via GOT rather than PLT stubs;
+#                                PLT stubs have no DWARF info and show as ??? in callgrind.
+#   -fno-builtin                 Keep stdlib calls (memcpy, strlen, etc.) visible in the graph
+#   -fno-optimize-sibling-calls  Suppress tail-call elimination (-O1 can still apply it)
 RUN ./configure \
     --enable-developer \
     --disable-verify-ptr \
     --with-raddbdir=/etc/freeradius \
-    CFLAGS="-g3 -O0 -fno-omit-frame-pointer -fno-inline -Dalways_inline= -fno-plt -fno-builtin -fno-optimize-sibling-calls" \
+    CFLAGS="-g3 -O1 -fno-omit-frame-pointer -fno-inline -Dalways_inline= -fno-plt -fno-builtin -fno-optimize-sibling-calls" \
     LDFLAGS="-fno-omit-frame-pointer"
 
 RUN make