[Openmp-commits] [PATCH] D93079: [OpenMP] Introduce an assumption to ignore possible external callers

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

Ah, this may be one of the things aomp changed for nvptx. We did something around linking but I've never looked into exactly what. In that case fine by me - it's a sharp edge, but I'd expect it to make a difference in benchmarks until whole program optimisation is online.

I think the strategy we want to converge on is equivalent to:

  N openmp source files -> bitcode -> llvm-link -> mlink-builtin-bitcode=nvptx.devicertl -> opt -> ptxas 

which interacts poorly with the compile/link separation. So presumably we should blend in the devicertl in an lld plugin, as that's when users expect the whole program to be available.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93079