[PATCH] D37631: [libFuzzer] Support using libc++

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added inline comments.


================
Comment at: lib/fuzzer/CMakeLists.txt:81
 
+if(CMAKE_CXX_COMPILER_ID MATCHES Clang AND COMPILER_RT_HAS_LIBCXX_SOURCES)
+  foreach(arch ${FUZZER_SUPPORTED_ARCH})
----------------
why do we need check for Clang here and below?


================
Comment at: test/fuzzer/lit.cfg:84
+
+config.substitutions.append(('%libcxx_c_compiler',
+      generate_compiler_cmd(is_cpp=False, libcxx=True, fuzzer_enabled=True)
----------------
Is any patch with tests which shows how this substitutions are going to be used?
I'd probably keep this part of change with actual tests using this.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D37631