[llvm] [Bolt] Fix issue 110407 (Support CREL) (PR #119150)

Vladislav Khmelevsky via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 18 03:37:45 PST 2024


yota9 wrote:

> > I think not only reading, but patching them is needed after binary processing
> 
> If I understand correctly, there is no need to update relocations metadata unless a second round of instrumentation/optimization is needed for the instrumented/optimized binary?
> 
> I can use this PR to optimize a clang built with CREL.

It seems I misunderstood and CREL is currently used for static relocations only. Then yes, the patch looks good to me, except for test absence :)

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


More information about the llvm-commits mailing list