This is an archive of the discontinued LLVM Phabricator instance.

Differential D104755

gn build: Only build the TSan runtime on 64-bit platforms.
ClosedPublic

Authored by pcc on Jun 22 2021, 2:45 PM.

Details

Reviewers
thakis
Commits
rG0439ba99031b: gn build: Only build the TSan runtime on 64-bit platforms.
Summary

TSan only supports 64-bit platforms.

Diff Detail

Event Timeline

pcc requested review of this revision.Jun 22 2021, 2:45 PM
pcc created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJun 22 2021, 2:45 PM
Harbormaster completed remote builds in B110514: Diff 353801.Jun 22 2021, 2:45 PM
thakis accepted this revision.Jun 22 2021, 5:40 PM
This revision is now accepted and ready to land.Jun 22 2021, 5:40 PM
This revision was landed with ongoing or failed builds.Jun 22 2021, 5:51 PM
Closed by commit rG0439ba99031b: gn build: Only build the TSan runtime on 64-bit platforms. (authored by pcc). · Explain Why
This revision was automatically updated to reflect the committed changes.
pcc added a commit: rG0439ba99031b: gn build: Only build the TSan runtime on 64-bit platforms..

Revision Contents

PathSize
llvm/
utils/
gn/
secondary/
compiler-rt/
lib/
4 lines

Diff 353832

llvm/utils/gn/secondary/compiler-rt/lib/BUILD.gn

group("lib") { group("lib") {
deps = [ "//compiler-rt/lib/profile" ] deps = [ "//compiler-rt/lib/profile" ]
if (current_os == "linux") { if (current_os == "linux") {
deps += [ "//compiler-rt/lib/msan" ] deps += [ "//compiler-rt/lib/msan" ]
} }
if (current_os != "win") { if (current_os != "win") {
deps += [ deps += [
"//compiler-rt/lib/asan", "//compiler-rt/lib/asan",
"//compiler-rt/lib/builtins", "//compiler-rt/lib/builtins",
"//compiler-rt/lib/tsan",
"//compiler-rt/lib/ubsan_minimal", "//compiler-rt/lib/ubsan_minimal",
] ]
if (current_cpu == "x64" || current_cpu == "arm64") {
deps += [ "//compiler-rt/lib/tsan" ]
}
} }
} }