[WIP] Enhance MCTS hybrid search features

[Copilot]

• Fix LTO linking error for GPUTuningParams::select_strategy() by making it inline in the header
• Verify the fix builds successfully
• Reply to the comment with the fix

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

---

Note

Introduces generated CMake artifacts to document a successful CodeQL build.

• Adds _codeql_build_dir containing CMakeCache.txt, CMakeConfigureLog.yaml, makefiles, progress files, and dependency info
• Confirms compiler/linker config and successful build of metalfish and metalfish_tests with LTO enabled

^{Written by Cursor Bugbot for commit 669b3df. This will update automatically on new commits. Configure here.}

[NripeshN]

bugbot run

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}

[cursor]

Build artifacts accidentally committed to repository

Medium Severity

The entire _codeql_build_dir/ directory containing CMake build artifacts from a CodeQL analysis run has been committed. These auto-generated files include CMakeCache.txt, Makefiles, dependency info, and configuration logs with hardcoded absolute paths to the CI runner environment (/home/runner/work/MetalFish/...). Build directories should be added to .gitignore, not version control.

Additional Locations (2)

• _codeql_build_dir/CMakeFiles/CMakeConfigureLog.yaml#L1-L328
• _codeql_build_dir/CMakeFiles/metalfish.dir/build.make#L1-L645

 

[Copilot]

Pull request overview

This PR fixes an LTO (Link-Time Optimization) linking error by converting GPUTuningParams::select_strategy() from an out-of-line member function to an inline member function defined directly in the header file.

Changes:

• Moved implementation of select_strategy() into the header as an inline function
• Removed the corresponding implementation from the .cpp file

Reviewed changes

Copilot reviewed 33 out of 35 changed files in this pull request and generated no comments.

File	Description
src/gpu/gpu_nnue_integration.h	Defines select_strategy() as inline with implementation in header
src/gpu/gpu_nnue_integration.cpp	Removes out-of-line implementation of select_strategy()
_codeql_detected_source_root	Adds CodeQL source root marker
_codeql_build_dir/*	Build system generated files from CodeQL analysis