Skip to content

Commit

Permalink
Update emscripten config settings to conform to the official emscript…
Browse files Browse the repository at this point in the history
…en toolchain.

Current Bazel config settings for Emscripten are specific to TensorFlow.js's custom Emscripten toolchain, but TFJS is switching to the official toolchain in [#4769](tensorflow/tfjs#4769). The official emscripten toolchain expects `crosstool_top` to be `//emscripten_toolchain:everything`.

PiperOrigin-RevId: 360787196
  • Loading branch information
xnnpack-bot committed Mar 4, 2021
1 parent fb0ab0b commit f69ac5d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -7438,21 +7438,21 @@ config_setting(

config_setting(
name = "emscripten",
values = {"crosstool_top": "//toolchain:emscripten"},
values = {"crosstool_top": "//emscripten_toolchain:everything"},
)

config_setting(
name = "emscripten_wasm",
values = {
"crosstool_top": "//toolchain:emscripten",
"crosstool_top": "//emscripten_toolchain:everything",
"cpu": "wasm",
},
)

config_setting(
name = "emscripten_wasmsimd",
values = {
"crosstool_top": "//toolchain:emscripten",
"crosstool_top": "//emscripten_toolchain:everything",
"cpu": "wasm",
"copt": "-msimd128",
},
Expand Down

0 comments on commit f69ac5d

Please sign in to comment.