[llvm-branch-commits] [llvm] [BOLT] Emit empty FDE for injected functions (PR #87967)

[Amir Ayupov via llvm-branch-commits]

aaupov wrote:

> > > Let's make `hasCFI()` return true for injected functions.
> > 
> > 
> > Given that this change increases the size of eh_frame section, should we make it dependent on `enable-bat`, i.e. when we expect to feed the binary back to BOLT?
> > Because otherwise this "fix" is a pure size regression.
> 
> Creating a proper FDE entry is the right thing to do regardless of BAT. How large is the regression?

The largest I've seen is 34M->39M (HHVM instrumentation).

https://github.com/llvm/llvm-project/pull/87967