[llvm] [llvm] Revise IDE folder structure (PR #89741)
Michael Kruse via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 24 04:20:22 PDT 2024
================
@@ -18,5 +18,5 @@ if (NOT WIN32 AND NOT CYGWIN)
# it is built before the plugin.
add_dependencies(InlineOrderPlugin intrinsics_gen)
add_dependencies(AnalysisTests InlineOrderPlugin)
- set_property(TARGET InlineOrderPlugin PROPERTY FOLDER "Tests/UnitTests/AnalysisTests")
+ set_property(TARGET InlineOrderPlugin PROPERTY FOLDER "Tests/Unittests/AnalysisTests")
----------------
Meinersbur wrote:
I seem having forgotten to remove this one, because the FOLDER is already set by `add_llvm_library`. The plugin is not added under WIN32 so I didn't notice. I updated this patch.
The case change was me trying to get the case consistent across projects before I moved to handling this inside `add_llvm_library`. With two folders differentiating only in case, the VS IDE (or CMake) either renames one of them to "UnitTests (2)", or throws an error when opening such a project.
https://github.com/llvm/llvm-project/pull/89741
More information about the llvm-commits
mailing list