diff options
| author | Brian Gesiak <modocache@gmail.com> | 2017-09-29 19:34:57 +0000 |
|---|---|---|
| committer | Brian Gesiak <modocache@gmail.com> | 2017-09-29 19:34:57 +0000 |
| commit | 73692018517748f4f3fbd64aabe60d2231a65d8d (patch) | |
| tree | 00e403c90637766add09d5164513ac811c43b753 /cmake/modules | |
| parent | b121e77d2ef4e7dd30fbbe9404315b5852bcd1c9 (diff) | |
| download | llvm-73692018517748f4f3fbd64aabe60d2231a65d8d.tar.gz | |
[CMake] Remove `CMAKE_.*_OUTPUT_DIRECTORY` (NFCI)
Summary:
Three `CMAKE_.*_OUTPUT_DIRECTORY` variables used to be set in CMake and
referenced in various other parts of the project. However, in r198205
chapuni added a note to "don't set them anymore", and any remaining
references to them were subsequently removed in r198316 and r199592.
Now that the variables are no longer used anywhere, remove them, along
with the comments advising against using them any longer.
Test Plan:
I ran `check-all` and confirmed the tests built and passed.
Reviewers: beanz, chapuni
Reviewed By: beanz
Subscribers: mgorny
Differential Revision: https://reviews.llvm.org/D38389
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@314550 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'cmake/modules')
| -rw-r--r-- | cmake/modules/AddLLVM.cmake | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/cmake/modules/AddLLVM.cmake b/cmake/modules/AddLLVM.cmake index 81e7211ef97..64ef479f30b 100644 --- a/cmake/modules/AddLLVM.cmake +++ b/cmake/modules/AddLLVM.cmake @@ -214,8 +214,6 @@ function(add_link_opts target_name) endfunction(add_link_opts) # Set each output directory according to ${CMAKE_CONFIGURATION_TYPES}. -# Note: Don't set variables CMAKE_*_OUTPUT_DIRECTORY any more, -# or a certain builder, for eaxample, msbuild.exe, would be confused. function(set_output_directory target) cmake_parse_arguments(ARG "" "BINARY_DIR;LIBRARY_DIR" "" ${ARGN}) |