[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