[Openmp-commits] [PATCH] D94573: [OpenMP] Drop the static library libomptarget-nvptx

[George Rokos via Phabricator via Openmp-commits]

grokos added a comment.

> One question is, why do we need two libraries? The only difference is, the static library contains omp_data.cu and the bitcode library doesn't. It's unclear why they were implemented in this way

It's been a very long time since then (6+ years), but if memory serves me well the reason we needed to build the static library in the early days was that a lot of systems lacked a CUDA compiler capable of emitting LLVM bitcode, so the static library was the only choice. Things have changed a lot in the meantime, so I guess the static library fallback is irrelevant nowadays.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94573