[PATCH] D45575: [scudo] Remove duplicate cmake flags

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 12 09:30:27 PDT 2018


cryptoad created this revision.
cryptoad added a reviewer: alekseyshl.
Herald added subscribers: Sanitizers, delcypher, mgorny.

Now that common options are propagated again for runtimes build with https://reviews.llvm.org/D45507,
the -f{data,function}-sections flags are now duplicates, remove them.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D45575

Files:
  lib/scudo/CMakeLists.txt


Index: lib/scudo/CMakeLists.txt
===================================================================
--- lib/scudo/CMakeLists.txt
+++ lib/scudo/CMakeLists.txt
@@ -4,7 +4,7 @@
 
 set(SCUDO_CFLAGS ${SANITIZER_COMMON_CFLAGS})
 # SANITIZER_COMMON_CFLAGS include -fno-builtin, but we actually want builtins!
-list(APPEND SCUDO_CFLAGS -fbuiltin -ffunction-sections -fdata-sections)
+list(APPEND SCUDO_CFLAGS -fbuiltin)
 append_rtti_flag(OFF SCUDO_CFLAGS)
 
 set(SCUDO_DYNAMIC_LIBS ${SANITIZER_COMMON_LINK_LIBS})


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D45575.142202.patch
Type: text/x-patch
Size: 505 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180412/ad664a36/attachment.bin>


More information about the llvm-commits mailing list