stella.stamenova added a comment. In https://reviews.llvm.org/D54567#1299992, @aprantl wrote:
> In https://reviews.llvm.org/D54567#1299989, @stella.stamenova wrote: > > > We should also remove LLDB_TEST_C_COMPILER and LLDB_TEST_CXX_COMPILER from > > the cmake files along with this change, otherwise, people will still expect > > them to work. > > > That would not be a good idea. There are several bots that are using these > flags. The change that Zachary is making is removing their usage, so after his change they would not do anything. If he ends up committing this change, these two properties (along with LLDB_DEFAULT_TEST_C_COMPILER and LLDB_TEST_USE_CUSTOM_C_COMPILER, etc.) should also go. https://reviews.llvm.org/D54567 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
