[Openmp-commits] [PATCH] D126701: [Libomptarget] Do not use retaining attributes for the static library

[Joseph Huber via Phabricator via Openmp-commits]

jhuber6 added a comment.

In D126701#3563526 <https://reviews.llvm.org/D126701#3563526>, @JonChesterfield wrote:

> Can we link the bitcode after openmp-opt? I was never keen on llvm emitting calls to functions that were linked in earlier and may have since been deleted. Optimising by runtime function name before linking them should be equivalent to what we have now and that renders the keepalive hack unnecessary.

You would need to link it after any optimizations that could potentially inline the functions, which somewhat defeats the purpose of doing this over the static library for now.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126701/new/

https://reviews.llvm.org/D126701