test: add RUN_PROFILE_TEST flag#178
Open
kilinchange wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
本 PR 调整了
scripts/run_models_and_profile.bash的构建与测试控制逻辑,主要目标是降低默认全量测试耗时,避免每次都编译并执行 profiler 测试。核心改动包括:
RUN_PROFILE_TEST配置项,用于控制是否运行 profiler 相关测试。test_config.json中普通 build / profiler build 两套重复配置合并为basic_compile_commands,配置文件只保留基础编译命令。RUN_CTEST注入BUILD_TEST=ON/OFFPROFILE_MODE=ONBUILD_TEST=OFF,避免额外编译 ctest 目标RUN_PROFILE_TEST=false时,脚本会跳过 profiler build 和对应的 profile 测试流程。这样配置层只描述基础编译方式,测试脚本负责根据运行开关派生实际构建模式,减少重复配置,也避免
RUN_CTEST/RUN_PROFILE_TEST与写死的 CMake 参数不一致。