compnerd added a comment.

@JDevlieghere I think that for LLDB, which this patch does not touch, the 
proper thing to do is to have 2 different paths with the ability to explicitly 
opt into the path, i.e. `LLDB_USE_PYTHON2:BOOL` and `LLDB_USE_PYTHON3:BOOL` 
with `CMakeDependentOption` to prevent the two from being set to `TRUE` 
simultaneously.  If two paths need to be supported, then you must have two 
paths.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78762/new/

https://reviews.llvm.org/D78762



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

Reply via email to