Skip to content

Commit 1ef97c7

Browse files
authored
Add target features section to bulk memory asm files (#22241)
Without this the feature might not be enabled when linked into the final binary. This fixes the lto2.test_dylink_syslibs_all which was broken by #22231 which enabled bigint, whicn in turn enabled bulk memory (by bumping the min browser version).
1 parent b06dc72 commit 1ef97c7

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

system/lib/libc/emscripten_memcpy_bulkmem.S

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,9 @@ _emscripten_memcpy_bulkmem:
1313
memory.copy 0, 0
1414
local.get 0
1515
end_function
16+
17+
.section .custom_section.target_features,"",@
18+
.int8 1
19+
.int8 43
20+
.int8 11
21+
.ascii "bulk-memory"

system/lib/libc/emscripten_memset_bulkmem.S

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,9 @@ _emscripten_memset_bulkmem:
1313
memory.fill 0
1414
local.get 0
1515
end_function
16+
17+
.section .custom_section.target_features,"",@
18+
.int8 1
19+
.int8 43
20+
.int8 11
21+
.ascii "bulk-memory"

0 commit comments

Comments
 (0)