maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[cmake] Use source-groups in Polly. 

Configure CMake to setup source-groups for Polly. Source groups
describe how source files should be organized in IDEs. By default, all
headers are dumped into one folder under PollyCore and all source files
into another. On disk, these files are organized into folders, but this
isn't reflected in the IDE. This change uses CMake source groups to have
the IDE reflect the on disk layout. This will make it easier to visualize
the project structure for users of Visual Studio and XCode

Patch by Christopher Tetreault <ctetreau@quicinc.com>

Reviewed By: Meinersbur, grosser

Differential Revision: https://reviews.llvm.org/D72117
This commit is contained in:
Christopher Tetreault 2020-01-07 14:15:07 -06:00 committed by Michael Kruse
parent 449ab10509
commit 76ee0de00c
2 changed files with 36 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
polly/cmake/polly_macros.cmake
View File

@ -86,3 +86,28 @@ function(target_enable_c99 _target)
endforeach()
endif()
endfunction()
# Recursive helper for setup_source_group. Traverse the file system and add
# source files matching the glob_expr to the prefix, recursing into
# subdirectories as they are encountered
function(setup_polly_source_groups_helper pwd prefix glob_expr)
file(GLOB children RELATIVE ${pwd} ${pwd}/*)
foreach(child ${children})
if (IS_DIRECTORY ${pwd}/${child})
setup_polly_source_groups_helper(${pwd}/${child}
"${prefix}\\${child}" ${glob_expr})
endif()
endforeach()
file(GLOB to_add ${pwd}/${glob_expr})
source_group(${prefix} FILES ${to_add})
endfunction(setup_polly_source_groups_helper)
# Set up source groups in order to nicely organize source files in IDEs
macro(setup_polly_source_groups src_root hdr_root)
# FIXME: The helper can be eliminated if the CMake version is increased
# to 3.8 or higher. If this is done, the TREE version of source_group can
# be used
setup_polly_source_groups_helper(${src_root} "Source Files" "*.cpp")
setup_polly_source_groups_helper(${hdr_root} "Header Files" "*.h")
endmacro(setup_polly_source_groups)

11
polly/lib/CMakeLists.txt
View File

@ -73,6 +73,17 @@ add_llvm_pass_plugin(Polly
set_target_properties(obj.Polly PROPERTIES FOLDER "Polly")
set_target_properties(Polly PROPERTIES FOLDER "Polly")
if (MSVC_IDE OR XCODE)
# Configure source groups for Polly source files. By default, in the IDE there
# will be a source and include folder. In the source folder will be all the
# source files in a flat list, and in the include folder will be all the
# headers in a flat list. Sets the CMake source_group for each folder such
# the organization of the sources and headers in the IDE matches how it is
# laid out on disk
setup_polly_source_groups(${CMAKE_CURRENT_LIST_DIR}
${CMAKE_CURRENT_LIST_DIR}/../include/polly)
endif()
# Create the library that can be linked into LLVM's tools and Polly's unittests.
# It depends on all library it needs, such that with
# LLVM_POLLY_LINK_INTO_TOOLS=ON, its dependencies like PollyISL are linked as