[Openmp-commits] [PATCH] D146925: [OpenMP] Fix building the second import library when built in the main llvm build root

Vadim Paretsky via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Mar 27 13:24:51 PDT 2023


vadikp-intel added a comment.

I will be good with this solution, but I think there could be a simpler one in the end.  While CMAKE does wants to see a source, that does not have to be a `cpp` (which is what's causing this need to unnecessarily feed the first step's library to satisfy `lib`'s attempt at `def` symbols resolution). There are other source types (e.g., a. `txt`) which will satisfy the source presence requirement but which CMAKE will filter out from the librarian generating the right command line for it.

I will submit a patch with this for review to https://reviews.llvm.org/D146894.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146925



More information about the Openmp-commits mailing list