[all-commits] [llvm/llvm-project] dd832c: [RuntimeDyld] Speedup resolution of relocations to...

[maksfb via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: dd832c7d3a7c808d76207a95a97ca14b3006874d
      https://github.com/llvm/llvm-project/commit/dd832c7d3a7c808d76207a95a97ca14b3006874d
  Author: Maksim Panchenko <maks at fb.com>
  Date:   2021-03-11 (Thu, 11 Mar 2021)

  Changed paths:
    M llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp

  Log Message:
  -----------
  [RuntimeDyld] Speedup resolution of relocations to external symbols

>From what I can tell, the loop inside applyExternalSymbolRelocations()
used to call getSymbolAddress(). After the JITSymbolResolver interface
redesign, the functionality has changed, and the loop should no longer
trigger repopulation of ExternalSymbolRelocations. If that's the case,
there is no need to update the loop iterator manually, and
ExternalSymbolRelocations can be cleared at once. This way, when there
are many external symbols in the program, the function runs much faster.

Reviewed By: lhames

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