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

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 06:37:34 PST 2023


sbc100 added a comment.

Also, I think it might be cleaner to simply generate the second function while the main function is being generated, rather then generate the long/unsplit function completely and then split it up.

Something like this phsudocode:

  for (r in relocations) {
    if (current_size_is_over_limit()) {
      generate_tail_call_to_new_function():
      switch_current_output_function_to_new_function();
    }
    generate_code_for_relocation(r);
  }

Then no splitting would be needed.


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

https://reviews.llvm.org/D140111



More information about the llvm-commits mailing list