experiments
/
STT-tensorflow
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix regression that omitted -DNDEBUG in opt builds after toolchain refactoring. 

Not adding -DNDEBUG leads to enabling assertions which makes LLVM often 10x slower.

PiperOrigin-RevId: 317349500
Change-Id: I52df6ab5013ad5a02101dd96f18220e054c3e94c
This commit is contained in:
A. Unique TensorFlower 2020-06-19 11:48:22 -07:00 committed by TensorFlower Gardener
parent 2417a15bf1
commit f805153a25
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
third_party/gpus/crosstool/cc_toolchain_config.bzl.tpl vendored
View File

@ -583,7 +583,11 @@ def _features(cpu, compiler, ctx):
), ),
], ],
), ),
feature(name = "opt"), feature(name = "disable-assertions"),
feature(
name = "opt",
implies = ["disable-assertions"],
),
feature(name = "fastbuild"), feature(name = "fastbuild"),
feature(name = "dbg"), feature(name = "dbg"),
feature(name = "supports_dynamic_linker", enabled = True), feature(name = "supports_dynamic_linker", enabled = True),