Phase 5: set_compile_options() で -Wall -Wextra -Werror を有効化 (#43)

[thawk105]

概要

issue #43 の段階的 -Werror 化、最終フェーズ (Phase 5 = 仕上げ)。Phase 1〜4 で 9 個の -Werror=<flag> クラスを 1 つずつ潰してきたが、Phase 5 ではその明示リストを set_compile_options() (= -Wall -Wextra -Werror) に置き換える。

cmake 変更

• cmake/ProtocolHelpers.cmake: ccbench_add_protocol() 内の target_compile_options(... -Werror=... ) 9 個の列を set_compile_options(${target}) 呼び出しに置き換え。
• CMakeLists.txt: set_compile_options() を使えるよう include(CompileOptions) を冒頭に hoist し、ccbench_common ターゲットにも適用。
• cc/silo/CMakeLists.txt: スタンドアロンの replay_test.exe にも適用。

set_compile_options() が有効化するのは -Wall -Wextra -Werror の 3 つだけ (cmake/CompileOptions.cmake)。-Wpedantic / -Wshadow / -Wconversion 等は含まない。

新規 warning と対処

-Wall -Wextra を ccbench_common と全プロトコルバイナリに適用したことで、Phase 1〜4 で潰した 9 クラス以外に 3 クラスが表面化した。いずれも小規模・機械的なので本 PR 内で対処:

ファイル	warning	対処
common/result.cc	-Wunused-parameter (3 件)	displayOzeAnalysisResult() の引数は #if ADD_ANALYSIS 時のみ使用 → 直下の displayAllResult() と同じく [[maybe_unused]] を付与
cc/silo/include/log.hh	-Wclass-memaccess (2 箇所)	LogRecord は std::string_view を持つ非 trivial 型。computeChkSum() のため padding 含め全バイトゼロ化する意図は正しいので、memset を static_cast<void *>(this) 経由に変更
include/tpcc/tpcc_util.hh	-Wstringop-overflow (26 件)	GCC 13 が fill_random() を random_string_detail() にインライン展開した際の interprocedural な誤検知。ループは size >= 8 の間しか回らず、呼び出し側は常に max_len + 1 バイトのバッファを渡すため overflow しない。ホットな init パスを悪化させず、説明コメント付きで局所的に suppress

test plan

GCC 13 (= CI と同じコンパイラ) でクリーンフルビルド:

cmake -B build -DCMAKE_BUILD_TYPE=Release -DENABLE_SANITIZER=OFF \
  -DCMAKE_CXX_COMPILER=g++-13 -DCMAKE_C_COMPILER=gcc-13 .
cmake --build build -j4

結果: exit 0 / warning 0 / error 0。全 34 プロトコルバイナリ + ccbench_common + replay_test.exe がビルド成功。

Closes #43