Skip to content

Commit 78b0e08

Browse files
committed
Run selective build CI test with more restrictions
Revert reason for #14487 was that internal tests were running with "-Werror,-Wunused-variable" option but not OSS tests
1 parent c9f46e2 commit 78b0e08

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

codegen/tools/CMakeLists.txt

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,17 @@ target_include_directories(
2424

2525
# Compile options
2626
target_compile_options(
27-
selective_build PUBLIC -Wno-deprecated-declarations -fPIC -frtti -fexceptions
27+
selective_build
28+
PUBLIC -Wno-deprecated-declarations
29+
-fPIC
30+
-frtti
31+
-fexceptions
32+
-Werror
33+
-Wunused-variable
34+
-Wno-unknown-argument
2835
)
36+
# We suppress -Wno-unknown-argument because our build system passes -fPIC for
37+
# Unix builds, but we also build on Windows where it's ignored
2938

3039
# Link against required libraries
3140
target_link_libraries(selective_build PRIVATE executorch_core program_schema)

examples/selective_build/test_selective_build.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -183,6 +183,12 @@ fi
183183
if [[ $1 == "cmake" ]];
184184
then
185185
cmake_install_executorch_lib $CMAKE_BUILD_TYPE
186+
187+
# Build selective_build module for model-based selection
188+
echo "Building selective_build Python module..."
189+
cmake --preset pybind -DCMAKE_BUILD_TYPE=Release -Bcmake-out .
190+
cmake --build cmake-out --target selective_build -j9
191+
186192
test_cmake_select_ops_in_list
187193
test_cmake_select_ops_in_yaml
188194
test_cmake_select_ops_in_model

0 commit comments

Comments
 (0)