[all-commits] [llvm/llvm-project] 7ef47d: [LLD] Increase thunk pass limit

Peter Smith via All-commits all-commits at lists.llvm.org
Mon Mar 13 03:06:40 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7ef47dd54e2d9f103986aec90aca554f634cc3e6
      https://github.com/llvm/llvm-project/commit/7ef47dd54e2d9f103986aec90aca554f634cc3e6
  Author: Peter Smith <peter.smith at arm.com>
  Date:   2023-03-13 (Mon, 13 Mar 2023)

  Changed paths:
    M lld/ELF/Writer.cpp

  Log Message:
  -----------
  [LLD] Increase thunk pass limit

In issue 61250 https://github.com/llvm/llvm-project/issues/61250 there is
an example of a program that takes 17 passes to converge, which is 2 more
than the current limit of 15. Analysis of the program shows a particular
section that is made up of many roughly thunk sized chunks of code ending
in a call to a symbol that needs a thunk. Due to the positioning of the
section, at each pass a subset of the calls go out of range of their
original thunk, needing a new one created, which then pushes more thunks
out of range. This process eventually stops after 17 passes.

This patch is the simplest fix for the problem, which is to increase
the pass limit. I've chosen to double it which should comfortably
account for future cases like this, while only taking a few more
seconds to reach the limit in case of non-convergence.

As discussed in the issue, there could be some additional work done
to limit thunk reuse, this would potentially increase the number of
thunks in the program but should speed up convergence.

Differential Revision: https://reviews.llvm.org/D145786




More information about the All-commits mailing list