Skip to content

Commit 6fd3fc9

Browse files
committed
remove -disable-internalize workaround during lto, which was removed from llvm, and appears no longer needed
1 parent 59586f3 commit 6fd3fc9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

emcc

+2-2
Original file line numberDiff line numberDiff line change
@@ -1264,8 +1264,8 @@ try:
12641264
# If we can LTO, do it before dce, since it opens up dce opportunities
12651265
if shared.Building.can_build_standalone() and llvm_lto and llvm_lto != 2 and shared.Building.can_use_unsafe_opts():
12661266
if not shared.Building.can_inline(): link_opts.append('-disable-inlining')
1267-
# do not internalize in std-link-opts - it ignores internalize-public-api-list - and add a manual internalize
1268-
link_opts += ['-disable-internalize'] + shared.Building.get_safe_internalize() + ['-std-link-opts']
1267+
# add a manual internalize with the proper things we need to be kept alive during lto
1268+
link_opts += shared.Building.get_safe_internalize() + ['-std-link-opts']
12691269
# execute it now, so it is done entirely before we get to the stage of legalization etc.
12701270
shared.Building.llvm_opt(final, pre_fastcomp_opts + link_opts)
12711271
if DEBUG: save_intermediate('lto', 'bc')

0 commit comments

Comments
 (0)