[PATCH] D140111: [lld][wasm] Split __wasm_apply_data_relocs when it exceeds the function size limit

Illia Malachyn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 15 10:33:59 PST 2022


k1nder10 added inline comments.


================
Comment at: lld/wasm/SplitApplyDataRelocsStrategy.cpp:40
+  const size_t pos = function_to_split_.rfind(
+      instruction_opcode, function_to_split_.size() - config->maxFunctionSize -
+                              skip_instruction_count);
----------------
This should be changed to work properly. Will upload a new patchset tomorrow.

const size_t pos = function_to_split_.rfind(instruction_opcode, config->maxFunctionSize - skip_instruction_count);


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140111/new/

https://reviews.llvm.org/D140111



More information about the llvm-commits mailing list