zturner added inline comments.

================
Comment at: cmake/modules/LLDBConfig.cmake:61
@@ +60,3 @@
+      # below, otherwise CMake will replace the whitespace with a semicolon in 
some contexts (which would stuff things up).
+      set (PYTHON_EXECUTABLE 
$<$<CONFIG:Debug>:${PYTHON_DEBUG_EXE}>$<$<NOT:$<CONFIG:Debug>>:${PYTHON_RELEASE_EXE}>)
+      set (PYTHON_LIBRARY 
$<$<CONFIG:Debug>:${PYTHON_DEBUG_LIB}>$<$<NOT:$<CONFIG:Debug>>:${PYTHON_RELEASE_LIB}>)
----------------
This line is hard to parse mentally, and I'm not sure I've seen this kind of 
nested generator expression.  I trust you when you say it's right, but can you 
explain what this does?


Repository:
  rL LLVM

http://reviews.llvm.org/D13234



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to