This is an archive of the discontinued LLVM Phabricator instance.

Differential D116510

[llvm][cmake] Normalize some indent
ClosedPublic

Authored by Ericson2314 on Jan 2 2022, 5:31 PM.

Details

Reviewers
mstorsjo
Commits
rGfc78b62cbada: [llvm][cmake] Normalize some indent
Summary
  • Change a stray tab to spaces
  • 4 not 2 spaces in second line of set like above.

Diff Detail

Event Timeline

Ericson2314 created this revision.Jan 2 2022, 5:31 PM
Herald added a subscriber: mgorny. · View Herald TranscriptJan 2 2022, 5:31 PM
Ericson2314 requested review of this revision.Jan 2 2022, 5:31 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 2 2022, 5:31 PM
Herald added a subscriber: llvm-commits. · View Herald Transcript
Ericson2314 updated this revision to Diff 396971.Jan 2 2022, 5:54 PM

Add more

Ericson2314 retitled this revision from [llvm][cmake] Change a stray tab to spaces to [llvm][cmake] Normalize some indent.Jan 2 2022, 5:55 PM
Ericson2314 edited the summary of this revision. (Show Details)
Ericson2314 mentioned this in D100810: [llvm] Use `GNUInstallDirs` to support custom installation dirs.Jan 2 2022, 6:04 PM
Harbormaster completed remote builds in B141263: Diff 396971.Jan 2 2022, 6:38 PM
mstorsjo accepted this revision.Jan 2 2022, 9:23 PM

LGTM

This revision is now accepted and ready to land.Jan 2 2022, 9:23 PM
This revision was landed with ongoing or failed builds.Jan 2 2022, 11:34 PM
Closed by commit rGfc78b62cbada: [llvm][cmake] Normalize some indent (authored by Ericson2314). · Explain Why
This revision was automatically updated to reflect the committed changes.
Ericson2314 added a commit: rGfc78b62cbada: [llvm][cmake] Normalize some indent.

Revision Contents

PathSize
llvm/
2 lines

Diff 396969

llvm/CMakeLists.txt

Show First 20 Lines • Show All 64 Lines • ▼ Show 20 Lines
# This allows an easy way of setting up a build directory for llvm and another # This allows an easy way of setting up a build directory for llvm and another
# one for llvm+clang+... using the same sources. # one for llvm+clang+... using the same sources.
set(LLVM_ALL_PROJECTS "clang;clang-tools-extra;compiler-rt;cross-project-tests;libc;libclc;libcxx;libcxxabi;libunwind;lld;lldb;mlir;openmp;polly;pstl") set(LLVM_ALL_PROJECTS "clang;clang-tools-extra;compiler-rt;cross-project-tests;libc;libclc;libcxx;libcxxabi;libunwind;lld;lldb;mlir;openmp;polly;pstl")
# The flang project is not yet part of "all" projects (see C++ requirements) # The flang project is not yet part of "all" projects (see C++ requirements)
set(LLVM_EXTRA_PROJECTS "flang") set(LLVM_EXTRA_PROJECTS "flang")
# List of all known projects in the mono repo # List of all known projects in the mono repo
set(LLVM_KNOWN_PROJECTS "${LLVM_ALL_PROJECTS};${LLVM_EXTRA_PROJECTS}") set(LLVM_KNOWN_PROJECTS "${LLVM_ALL_PROJECTS};${LLVM_EXTRA_PROJECTS}")
set(LLVM_ENABLE_PROJECTS "" CACHE STRING set(LLVM_ENABLE_PROJECTS "" CACHE STRING
"Semicolon-separated list of projects to build (${LLVM_KNOWN_PROJECTS}), or \"all\".") "Semicolon-separated list of projects to build (${LLVM_KNOWN_PROJECTS}), or \"all\".")
foreach(proj ${LLVM_ENABLE_PROJECTS}) foreach(proj ${LLVM_ENABLE_PROJECTS})
if (NOT proj STREQUAL "all" AND NOT proj STREQUAL "llvm" AND NOT "${proj}" IN_LIST LLVM_KNOWN_PROJECTS) if (NOT proj STREQUAL "all" AND NOT proj STREQUAL "llvm" AND NOT "${proj}" IN_LIST LLVM_KNOWN_PROJECTS)
MESSAGE(FATAL_ERROR "${proj} isn't a known project: ${LLVM_KNOWN_PROJECTS}") MESSAGE(FATAL_ERROR "${proj} isn't a known project: ${LLVM_KNOWN_PROJECTS}")
endif() endif()
endforeach() endforeach()
foreach(proj "libcxx" "libcxxabi" "libunwind") foreach(proj "libcxx" "libcxxabi" "libunwind")
if (${proj} IN_LIST LLVM_ENABLE_PROJECTS) if (${proj} IN_LIST LLVM_ENABLE_PROJECTS)
message(WARNING "Using LLVM_ENABLE_PROJECTS=${proj} is deprecated now, please use -DLLVM_ENABLE_RUNTIMES=${proj} or " message(WARNING "Using LLVM_ENABLE_PROJECTS=${proj} is deprecated now, please use -DLLVM_ENABLE_RUNTIMES=${proj} or "
▲ Show 20 Lines • Show All 1,146 Lines • Show Last 20 Lines