[objcopy][NFC] Add rules to cmake to put files under specific folders.

[avl]

Summary

This patch adds rules to cmake to put files under specific folders.

It allows to have files for different formats(which are located in different
subdirectories) be displayed in different subfolders of VS IDE solution.

Depends on D114429

[jhenderson]

llvm/lib/Object/CMakeLists.txt

2

Nit: delete this blank line.

3

The backslashes are surprising to me. Can this be forward slashes, like a cross-platform file path?

4

I think you want to use the REGULAR_EXPRESSION rather than FILES, so that you don't need to explicitly specify each source file in the list.

This should allow you to avoid the need for the macros below too.

93–96

Do you need these explicitly?

[avl]

llvm/lib/Object/CMakeLists.txt

3

If forward slashes are used then Visual Studio shows "ObjCopy" not as subfolder of "Header Files" but as another folder on the same level as "Header Files".

[avl]

addressed comments.

[avl]

llvm/lib/Object/CMakeLists.txt

93–96

yes, we need these. If we do not specify include paths here then corresponding .h files would not be shown in the "Header Files" folder.

[jhenderson]

LGTM.

llvm/lib/Object/CMakeLists.txt

3

That's bizarre, but okay.

93–96

Interesting. That suggests the add_llvm_component_library function doesn't recurse down the tree, which it intuitively should (possibly auto-adding the source groups along the way). Anyway, not an issue with this patch.

[avl]

rebased.

[jhenderson]

Thanks, much appreciated. LGTM again!

llvm/lib/Object/CMakeLists.txt

+source_group("Header Files\\ObjCopy" REGULAR_EXPRESSION
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/.*.h
+)
+
+source_group("Header Files\\ObjCopy\\COFF" REGULAR_EXPRESSION
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/COFF/.*.h
+)
+
+source_group("Header Files\\ObjCopy\\ELF" REGULAR_EXPRESSION
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/ELF/.*.h
+)
+
+source_group("Header Files\\ObjCopy\\MachO" REGULAR_EXPRESSION
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/MachO/.*.h
+)
+
+source_group("Header Files\\ObjCopy\\wasm" REGULAR_EXPRESSION
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/wasm/.*.h
+)
+
+source_group("Source Files\\ObjCopy" REGULAR_EXPRESSION
+  ObjCopy/.*.cpp
+)
+
+source_group("Source Files\\ObjCopy\\COFF" REGULAR_EXPRESSION
+  ObjCopy/COFF/.*.cpp
+)
+
+source_group("Source Files\\ObjCopy\\ELF" REGULAR_EXPRESSION
+  ObjCopy/ELF/.*.cpp
+)
+
+source_group("Source Files\\ObjCopy\\MachO" REGULAR_EXPRESSION
+  ObjCopy/MachO/.*.cpp
+)
+
+source_group("Source Files\\ObjCopy\\wasm" REGULAR_EXPRESSION
+  ObjCopy/wasm/.*.cpp
+)
+
 add_llvm_component_library(LLVMObject
   Archive.cpp
   ArchiveWriter.cpp
   Binary.cpp
   COFFImportFile.cpp
   COFFModuleDefinition.cpp
   COFFObjectFile.cpp
   Decompressor.cpp
add_llvm_component_library(LLVMObject
   ObjCopy/MachO/Object.cpp
   ObjCopy/wasm/Object.cpp
   ObjCopy/wasm/Reader.cpp
   ObjCopy/wasm/Writer.cpp
   ObjCopy/wasm/WasmObjcopy.cpp
 
   ADDITIONAL_HEADER_DIRS
   ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/COFF
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/ELF
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/MachO
+  ${LLVM_MAIN_INCLUDE_DIR}/llvm/Object/ObjCopy/wasm
 
   DEPENDS
   intrinsics_gen
   llvm_vcsrevision_h
 
   LINK_COMPONENTS
   BitReader
   Core
   MC
   BinaryFormat
   MCParser
   Support
   TextAPI
   )