Skip to content

Commit 312c78a

Browse files
akriegerfacebook-github-bot
authored andcommitted
Force -O3 for executorch op_div.cpp in "clang 17" also (#13660)
Summary: 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 312c78a

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)