Skip to content

Commit 1548bb1

Browse files
committed
Force -O3 for executorch op_div.cpp in "clang 17" also (#13660)
Summary: Pull Request resolved: #13660 Some platforms use clang 19 under the hood but are setting the clang 17 constraint for bad reasons. Reviewed By: swolchok Differential Revision: D80980948
1 parent d7fd78b commit 1548bb1

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

shim_et/xplat/executorch/kernels/optimized/op_registration_util.bzl

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -174,9 +174,10 @@ OPTIMIZED_ATEN_OPS = (
174174
# is not sufficient to avoid it.
175175
compiler_flags = [] if runtime.is_oss else select({
176176
"DEFAULT": [],
177-
"ovr_config//toolchain/clang/constraints:19": select({
177+
"ovr_config//os:android": select({
178178
"DEFAULT": [],
179-
"ovr_config//os:android": ["-O3"],
179+
"ovr_config//toolchain/clang/constraints:17": ["-O3"],
180+
"ovr_config//toolchain/clang/constraints:19": ["-O3"],
180181
}),
181182
}),
182183
deps = [

0 commit comments

Comments
 (0)