[PATCH] D28359: [compiler-rt] Use common static library for sanitizer_common.

Mike Aizatsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 5 12:39:30 PST 2017


aizatsky added inline comments.


================
Comment at: lib/stats/CMakeLists.txt:19
+    STATIC
+    ARCHS ${SANITIZER_COMMON_SUPPORTED_ARCH}
+    OS ${SANITIZER_COMMON_SUPPORTED_OS}
----------------
mpividori wrote:
> aizatsky wrote:
> > This one doesn't have RTSanitizerCommon & RTSanitizerCommonLibc. Is this intended?
> @aizatsky Yes, this is intended. I remove that from from sanitizer static libraries, and move that code to `san_common` static library.
But SHARED version needs them (above)?


Repository:
  rL LLVM

https://reviews.llvm.org/D28359





More information about the llvm-commits mailing list