[PATCH] D85626: [tools][remarks-shlib] Don't build libRemarks.so without PIC

[Rainer Orth via Phabricator via llvm-commits]

ro added inline comments.


================
Comment at: llvm/tools/remarks-shlib/CMakeLists.txt:1-2
-set(LLVM_LINK_COMPONENTS
-  Remarks
-  )
+# Building shared libraries requires PIC objects.
+if(LLVM_ENABLE_PIC)
+
----------------
lebedev.ri wrote:
> Is this the only LLVM shared library that was missing this `if(LLVM_ENABLE_PIC)` check?
> If not, this comment is misleading, since clearly it worked so far everywhere else.
No, `LLVMPolly.so` had the same issue: fixed in D85627.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85626