[llvm-branch-commits] [llvm] release/20.x: [llvm-objcopy] Apply encryptable offset to first segment, not section (#130517) (PR #131398)

[Daniel Rodríguez Troitiño via llvm-branch-commits]

drodriguez wrote:

> @drodriguez What do you think about merging this PR to the release branch?

@nikic I imagine this is you asking and not a sentient robot? :D

I think the fix is safe enough for cherry-picking into the release branch. A couple of people in the issue, PR, and related issues I have found had confirmed it works for them. Not sure if it classifies as important enough for cherry-picking into the release branch, but it is true that creates binaries that might be rejected from some tooling (but not the dynamic loaded itself in my experience).

In summary: not opposed to cherry-picking into the release branch, but also I will not be offended if this is not considered important enough.

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