[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-10-03 Thread Alexander Richardson via Phabricator via cfe-commits
arichardson added a comment. In case anyone else runs into this: It appears this change somehow broke incremental builds with `-DLLVM_ENABLE_RUNTIMES=libunwind` (even after deleting CMakeCache.txt in the main build dir): -- Using libunwind testing configuration:

[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-09-30 Thread Louis Dionne via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa48f018bb7d8: [runtimes] Remove all traces of the legacy testing configuration system (authored by ldionne). Repository: rG LLVM Github Monorepo

[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-09-30 Thread Louis Dionne via Phabricator via cfe-commits
ldionne updated this revision to Diff 464351. ldionne added a comment. Add release note. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134650/new/ https://reviews.llvm.org/D134650 Files: clang/cmake/caches/CrossWinToARMLinux.cmake

[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-09-30 Thread Louis Dionne via Phabricator via cfe-commits
ldionne added inline comments. Comment at: clang/cmake/caches/CrossWinToARMLinux.cmake:163 set(RUNTIMES_${TOOLCHAIN_TARGET_TRIPLE}_COMPILER_RT_EMULATOR "\\\"${Python3_EXECUTABLE}\\\" \\\"${LLVM_PROJECT_DIR}/llvm/utils/remote-exec.py\\\" --execdir %%T

[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-09-30 Thread Louis Dionne via Phabricator via cfe-commits
ldionne updated this revision to Diff 464234. ldionne added a comment. Address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134650/new/ https://reviews.llvm.org/D134650 Files: clang/cmake/caches/CrossWinToARMLinux.cmake

[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-09-29 Thread Vlad Vereschaka via Phabricator via cfe-commits
vvereschaka added inline comments. Comment at: clang/cmake/caches/CrossWinToARMLinux.cmake:163 set(RUNTIMES_${TOOLCHAIN_TARGET_TRIPLE}_COMPILER_RT_EMULATOR "\\\"${Python3_EXECUTABLE}\\\" \\\"${LLVM_PROJECT_DIR}/llvm/utils/remote-exec.py\\\" --execdir %%T

[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-09-29 Thread Louis Dionne via Phabricator via cfe-commits
ldionne updated this revision to Diff 464060. ldionne added a comment. Rebase on main. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134650/new/ https://reviews.llvm.org/D134650 Files: clang/cmake/caches/CrossWinToARMLinux.cmake

[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-09-29 Thread Louis Dionne via Phabricator via cfe-commits
ldionne added a subscriber: vvereschaka. ldionne added a comment. @vvereschaka Can you please confirm that the CMake cache changes look good to you? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134650/new/ https://reviews.llvm.org/D134650

[PATCH] D134650: [runtimes] Remove all traces of the legacy testing configuration system

2022-09-26 Thread Louis Dionne via Phabricator via cfe-commits
ldionne created this revision. Herald added subscribers: libcxx-commits, mstorsjo, arichardson. Herald added projects: libunwind, All. Herald added a reviewer: libunwind. ldionne requested review of this revision. Herald added projects: clang, libc++, libc++abi. Herald added a subscriber: