[all-commits] [llvm/llvm-project] 0977a7: [BOLT] Skip FDE emission for patch functions (#136...

[Maksim Panchenko via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0977a7130b291304dae5cfd23575ea3a4b9922ed
      https://github.com/llvm/llvm-project/commit/0977a7130b291304dae5cfd23575ea3a4b9922ed
  Author: Maksim Panchenko <maks at fb.com>
  Date:   2025-04-17 (Thu, 17 Apr 2025)

  Changed paths:
    M bolt/lib/Core/BinaryEmitter.cpp
    M bolt/test/AArch64/lite-mode.s

  Log Message:
  -----------
  [BOLT] Skip FDE emission for patch functions (#136224)

Patch functions are used to fix instructions in the original code, i.e.,
they are not functions in a traditional sense, but rather pieces of
emitted code that are embedded into real functions.

We used to emit FDEs for all functions, including patch functions.
However, FDEs for patches are not only unnecessary, but they can lead to
problems with libraries and runtimes that consume FDEs, e.g. C++
exception handling runtime.

Note that we use named patches to fix function entry points and in that
case they behave more like regular functions. Thus we issue FDEs for
those.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications