| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| |
| |
| | |
f6c21894 Modules: Rename internal platform-specific compiler determination modules
0c7951a9 cmLocalGenerator: Consolidate conditions in AddArchitectureFlags
|
| |
| |
| |
| | |
Make room for possible future non-Apple architecture flags.
|
|\ \
| | |
| | |
| | |
| | | |
f500a784 Fix crash on $<TARGET_PROPERTY:...,LOCATION> genex (#16134)
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Policy CMP0026 deprecated the LOCATION property, and we have long
provided a $<TARGET_FILE:...> generator expression. However, if
a project tries to use $<TARGET_PROPERTY:...,LOCATION> we should
at least not crash.
The compatibility implementation of the LOCATION property uses
cmGlobalGenerator::CreateGenerationObjects to create the structures
needed to evaluate the property before generation starts. The
implementation assumed that accessing the property could only be done
during configuration (via the typical get_property command use case).
The $<TARGET_PROPERTY:...,LOCATION> genex causes the LOCATION property
to be accessed during generation. Calling CreateGenerationObjects
during generation blows away all the objects currently being used for
generation and is not safe. Add a condition to call it only when
configuration is not finished.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
c6220de2 Use the empty() method to check for emptyness.
|
| | |/
| |/|
| | |
| | |
| | | |
Apply fix-its from clang-tidy's readability-container-size-empty
checker.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
7f6b8d33 Simplify boolean expressions
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Use clang-tidy's readability-simplify-boolean-expr checker.
After applying the fix-its, revise all changes *very* carefully.
Be aware of false positives and invalid changes.
|
| |/ /
|/| | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
2175e5bf cmGlobalGenerator: Make IsMultiConfig() const
|
| |/ / |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
18bfbc97 Add option to control 'bin' directory of CMake's own installation (#16076)
|
| |/ /
| | |
| | |
| | |
| | | |
Add a `CMAKE_BIN_DIR` cache entry to CMake's own build configuration.
Add a `--bindir` option to the `bootstrap` script to set it.
|
|/ / |
|
| | |
|
|/ |
|
|\
| |
| |
| |
| |
| | |
bd4fef64 cmSourceFileLocation: Fix typo in comment
814e774e cmSearchPath: Fix typo in comment
|
| | |
|
| | |
|
|/ |
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| |
| | |
27ead963 Remove unnecessary local copies.
618fb23f Pass arguments that are not modified as const&.
|
| |
| |
| |
| |
| |
| |
| | |
Use clang-tidy's performance-unnecessary-copy-initialization checker.
After applying the fix-its (which turns the copies into const&), revise
the changes and see whether the copies can be removed entirely by using
the original instead.
|
| |
| |
| |
| |
| |
| |
| | |
Use clang-tidy's performance-unnecessary-value-param checker to find
value parameter declarations of expensive to copy types that are not
modified inside the function. Ignore findings in kwsys.
After applying the fix-its, manually change `const T&` to `T const&`.
|
|\ \
| | |
| | |
| | |
| | | |
1b2bb933 Remove redundant c_str() calls.
|
| |/
| |
| |
| |
| | |
Run clang-tidy's readability-redundant-string-cstr checker.
Ignore findings in kwsys.
|
|/ |
|
|\
| |
| |
| |
| |
| | |
d256ba07 try_compile: Optionally forward custom platform variables to test project
fb4791b3 cmCoreTryCompile: Refactor forwarding of variables to test project
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a `CMAKE_TRY_COMPILE_PLATFORM_VARIABLES` variable to specify a list
of custom variables to be forwarded to a `try_compile` test project.
This will be useful for platform information modules or toolchain files
to forward some platform-specific set of variables from the host project
(perhaps set in its cache) to the test project so that it can build the
same way.
|
| |
| |
| |
| |
| |
| |
| | |
De-duplicate the logic that constructs the cmake `-D` flag used to pass
variables into the test project cache. Also subsume variables that were
propagated by generating `set()` commands in the project and pass them
as cache entries instead.
|
|\ \
| | |
| | |
| | |
| | | |
75e3e0d3 cmVariableWatch: Fix potential memory leak
|
| |/
| |
| |
| |
| | |
Teach cmVariableWatch::AddWatch to own the Pair it allocates until
it needs to pass ownership to WatchMap.
|
|\ \
| | |
| | |
| | |
| | | |
3acc29fc CPack/DragNDrop: Optionally disable `/Applications` symlink
|
| |/ |
|
|\ \
| | |
| | |
| | |
| | | |
8e801eb5 cmSystemTools: Fix indentation typo
|
| | | |
|
| |/
|/| |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
5784747d Improve string find: prefer character overloads.
5cec953e Use std::replace for replacing chars in strings.
2a1a2033 cmExtraEclipseCDT4Generator: use std::replace.
34bc6e1f cmCTestScriptHandler: don't call find repeatedly.
|
| | |
| | |
| | |
| | |
| | | |
Apply fix-its from clang-tidy's performance-faster-string-find checker.
Ignore findings in kwsys.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Find uses of `cmSystemTools::ReplaceString` where both `replace` and
`with` are string literals with a size of one.
Automate with:
git grep -l ReplaceString | xargs sed -i "s|cmSystemTools::ReplaceString(\([^,]*\), \"\(.\)\", \"\(.\)\");|std::replace(\1.begin(), \1.end(), '\2', '\3');|g"
git grep -l ReplaceString | xargs sed -i "s|cmSystemTools::ReplaceString(\([^,]*\), \"\(.\)\", \"\\\\\\\\\");|std::replace(\1.begin(), \1.end(), '\2', '\\\\\\\\');|g"
git grep -l ReplaceString | xargs sed -i "s|cmSystemTools::ReplaceString(\([^,]*\), \"\\\\\\\\\", \"\(.\)\");|std::replace(\1.begin(), \1.end(), '\\\\\\\\', '\2');|g"
|
| | | |
|
| | |
| | |
| | |
| | | |
Also, prefer the character overload.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
c1340827 Add a variable to specify language-wide system include directories
44199097 cmMakefile: Optimize AddSystemIncludeDirectories for empty set
a896043b GHS: Compute include directories consistently with other generators
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Create a `CMAKE_<LANG>_STANDARD_INCLUDE_DIRECTORIES` variable to specify
system include directories for for `<LANG>` compiler command lines.
This plays a role for include directories as the existing
`CMAKE_<LANG>_STANDARD_LIBRARIES` variable does for link libraries.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Do not bother looping over all targets if we have no system include
directories to add anyway.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
All generators use cmLocalGenerator::GetIncludeDirectories to construct
the final list of include directories for a target.
|