[compiler-rt] r317103 - [scudo] Fix standlone build -lrt requirement

Kostya Kortchinsky via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 1 10:00:26 PDT 2017


Author: cryptoad
Date: Wed Nov  1 10:00:26 2017
New Revision: 317103

URL: http://llvm.org/viewvc/llvm-project?rev=317103&view=rev
Log:
[scudo] Fix standlone build -lrt requirement

Summary:
The split in D39461 introduced separate C++ flags, but `cxx_flags` needs `-lrt` as well for the standalone build.


Reviewers: alekseyshl

Reviewed By: alekseyshl

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D39497

Modified:
    compiler-rt/trunk/test/scudo/lit.cfg

Modified: compiler-rt/trunk/test/scudo/lit.cfg
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/scudo/lit.cfg?rev=317103&r1=317102&r2=317103&view=diff
==============================================================================
--- compiler-rt/trunk/test/scudo/lit.cfg (original)
+++ compiler-rt/trunk/test/scudo/lit.cfg Wed Nov  1 10:00:26 2017
@@ -28,12 +28,13 @@ c_flags = ([config.target_cflags] +
            "-UNDEBUG",
            "-ldl",
            "-Wl,--gc-sections"])
-cxx_flags = (c_flags + config.cxx_mode_flags + ["-std=c++11"])
 
 # Android doesn't want -lrt.
 if not config.android:
   c_flags += ["-lrt"]
 
+cxx_flags = (c_flags + config.cxx_mode_flags + ["-std=c++11"])
+
 def build_invocation(compile_flags):                                            
   return " " + " ".join([config.compile_wrapper, config.clang] + compile_flags) + " "                   
 




More information about the llvm-commits mailing list