Skip to content

Commit 56f2661

Browse files
authored
Revert "using lld linker" (#1008)
Revert "using lld linker (#930)" This reverts commit f972ad9.
1 parent 99c1044 commit 56f2661

File tree

9 files changed

+6
-60
lines changed

9 files changed

+6
-60
lines changed

docker/dev/Dockerfile

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,10 @@ RUN apt-get update -y && apt-get install -y git curl vim libssl-dev wget unzip o
2020
RUN bash /install/scripts/install-cmake.sh
2121
RUN bash /install/scripts/install-bazel.sh
2222
RUN bash /install/scripts/install-python.sh
23-
RUN bash /install/scripts/install-llvm_toolchain.sh
2423

2524
ARG DEVICE=cu110
2625
RUN bash /install/scripts/install-tensorrt.sh
2726
RUN bash /install/scripts/install-cudnn.sh
2827

29-
ENV PATH="/usr/local/llvm_toolchain/bin:/opt/cmake/bin:${PATH}"
30-
ENV LD_LIBRARY_PATH="/usr/local/llvm_toolchain/lib:/usr/local/TensorRT/lib/:/usr/local/cuda/lib64/:${LD_LIBRARY_PATH}"
28+
ENV PATH="/opt/cmake/bin:${PATH}"
29+
ENV LD_LIBRARY_PATH="/usr/local/TensorRT/lib/:/usr/local/cuda/lib64/:${LD_LIBRARY_PATH}"

docker/dev/Dockerfile.aarch64

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,8 +84,7 @@ RUN apt-get -y update && \
8484
zip \
8585
zlib1g-dev \
8686
openjdk-11-jdk \
87-
patchelf \
88-
lld
87+
patchelf
8988

9089
RUN update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-10 1 && \
9190
update-alternatives --install /usr/bin/g++ g++ /usr/bin/g++-10 1 && \

docker/scripts/install-llvm_toolchain.sh

Lines changed: 0 additions & 37 deletions
This file was deleted.

pytorch_blade/bazel_build.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -152,8 +152,6 @@ def __init__(self, *args, **kwargs):
152152
if running_on_ci():
153153
self.configs += ["--config=ci_build"]
154154

155-
self.configs += [" --linkopt='-fuse-ld=gold -Xlinker --stub-group-size -Xlinker 10000000'"]
156-
157155
root_dir = os.path.join(os.path.dirname(os.path.abspath(__file__)), os.pardir)
158156
self.configs += [
159157
build_tao_compiler_add_flags_platform_alibaba_cached(root_dir, ""),

scripts/python/common_setup.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -180,15 +180,6 @@ def which(cmd):
180180
raise Exception("failed to find command: " + cmd)
181181
return found
182182

183-
def extra_link_flags(force_gold=True):
184-
from distutils.spawn import find_executable
185-
if find_executable("ld.lld"):
186-
return " --linkopt='-fuse-ld=lld -Xlinker -lm'"
187-
else:
188-
if force_gold:
189-
return " --linkopt='-fuse-ld=gold -Xlinker --stub-group-size -Xlinker 10000000'"
190-
else:
191-
return " --linkopt='-Xlinker --stub-group-size -Xlinker 10000000'"
192183

193184
@contextmanager
194185
def cwd(path):

scripts/python/tao_build.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@
4242
add_ral_link_if_not_exist,
4343
logger,
4444
which,
45-
extra_link_flags,
4645
running_on_ci,
4746
ci_build_flag,
4847
remote_cache_token,
@@ -281,7 +280,6 @@ def configure(root, args):
281280
@time_stage()
282281
def build_tao_compiler(root, args):
283282
BAZEL_BUILD_CMD = "bazel build --verbose_failures --experimental_multi_threaded_digest --define framework_shared_object=false" + ci_build_flag()
284-
BAZEL_BUILD_CMD += extra_link_flags()
285283
TARGET_TAO_COMPILER_MAIN = "//decoupling:tao_compiler_main"
286284
TARGET_DISC_OPT = "//mlir/disc:disc-opt"
287285
TARGET_DISC_REPLAY = "//mlir/disc/tools/disc-replay:disc-replay-main"
@@ -356,8 +354,6 @@ def test_tao_compiler(root, args):
356354
BAZEL_TEST_CMD = "bazel test --experimental_multi_threaded_digest --define framework_shared_object=false --test_timeout=600 --javabase=@bazel_tools//tools/jdk:remote_jdk11"
357355
BAZEL_TEST_CMD += ci_build_flag()
358356
BAZEL_BUILD_CMD += ci_build_flag()
359-
BAZEL_TEST_CMD += extra_link_flags()
360-
BAZEL_BUILD_CMD += extra_link_flags()
361357
if running_on_ci():
362358
# NOTE: using the lower parallel jobs on CI host to avoid OOM
363359
BAZEL_TEST_CMD += " --jobs=10 --test_output=errors"
@@ -476,7 +472,6 @@ def tao_bridge_bazel_config(args):
476472
bazel_config += " --config=disc_mkldnn"
477473
if args.platform_alibaba:
478474
bazel_config += " --config=platform_alibaba"
479-
bazel_config += extra_link_flags()
480475
return bazel_config
481476

482477
@time_stage()

tao/.bazelrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ build:disc --config=release_base
3737

3838
build:disc_cpu --config=disc --cxxopt=-DTAO_CPU_ONLY
3939
build:disc_x86 --config=disc_cpu --config=release_cpu_linux --cxxopt=-DTAO_X86 --define disc_x86=true
40-
build:disc_aarch64 --config=disc_cpu --cxxopt=-DTAO_AARCH64 --define disc_aarch64=true
40+
build:disc_aarch64 --config=disc_cpu --cxxopt=-DTAO_AARCH64 --define disc_aarch64=true --linkopt="-Xlinker --stub-group-size -Xlinker 10000000"
4141
build:disc_cuda --config=disc --config=cuda
4242
build:disc_dcu --config=disc --config=dcu
4343
build:disc_rocm --config=disc --config=rocm

tao_compiler/.bazelrc.user

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ build:ci_build --noshow_loading_progress --show_progress_rate_limit=600 --jobs=1
33
build:disc --define framework_shared_object=false --experimental_multi_threaded_digest
44
build:disc_cpu --config=disc --cxxopt=-DTAO_CPU_ONLY
55
build:disc_x86 --config=disc_cpu --config=release_cpu_linux --cxxopt=-DTAO_X86 --define disc_x86=true
6-
build:disc_aarch64 --config=disc_cpu --config=mkl_aarch64 --cxxopt=-DTAO_AARCH64 --define disc_aarch64=true
6+
build:disc_aarch64 --config=disc_cpu --config=mkl_aarch64 --cxxopt=-DTAO_AARCH64 --define disc_aarch64=true --linkopt="-Xlinker --stub-group-size -Xlinker 10000000"
77
build:disc_cuda --config=disc --config=cuda
88
build:disc_dcu --config=disc --config=dcu
99

tensorflow_blade/.bazelrc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ build:disc_aarch64 --config=disc_cpu
6767
build:disc_aarch64 --config=mkl_aarch64
6868
build:disc_aarch64 --cxxopt=-DTAO_AARCH64
6969
build:disc_aarch64 --define disc_aarch64=true
70+
build:disc_aarch64 --linkopt="-Xlinker --stub-group-size -Xlinker 10000000"
7071
build:disc_aarch64 --action_env BUILD_WITH_AARCH64=1
7172
build:disc_cuda --config=disc --config=cuda
7273
build:disc_dcu --config=disc --config=dcu

0 commit comments

Comments
 (0)