diff options
author | Tobias Grosser <tobias@grosser.es> | 2016-05-04 20:04:40 +0000 |
---|---|---|
committer | Tobias Grosser <tobias@grosser.es> | 2016-05-04 20:04:40 +0000 |
commit | 3b3261693ef4a8b8f08657cff92eb6cb58d2fdba (patch) | |
tree | 67f6518164dab0884250999df0c4a267478f8887 /lldb/packages/Python/lldbsuite/test/python_api/value | |
parent | 6b5c89262adac4943f830c943d27683bdd176e73 (diff) | |
download | bcm5719-llvm-3b3261693ef4a8b8f08657cff92eb6cb58d2fdba.tar.gz bcm5719-llvm-3b3261693ef4a8b8f08657cff92eb6cb58d2fdba.zip |
cmake: Prefix Polly options with LLVM_ to avoid variable shadowing
Summary:
Before this change certain Polly variables have been used both as user-facing
CACHED cmake variables as well as uncached internal variables. Even though
this seems to have worked OK in practice, the behavior only worked due to
one variable shadowing the other. This behavior has been found confusing.
To make the use of cmake variables more clear we now prefix the cached, user
facing variables with LLVM_ as it is common habit for LLVM options and also
moved the _POLLY_ term to the beginning to ensure related options are sorted
after each other. The variables that control the behavior of LLVM/Polly are then
set by forwarding the values set in the user facing option variables.
As a result, Polly is now enabled with LLVM_POLLY_BUILD instead of BUILD_POLLY
and the linking behavior of Polly is controlled with LLVM_POLLY_LINK_INTO_TOOLS
instead of LINK_POLLY_INTO_TOOLS.
Reviewers: bogner, Meinersbur
Subscribers: pollydev, llvm-commits
Differential Revision: http://reviews.llvm.org/D19907
llvm-svn: 268537
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api/value')
0 files changed, 0 insertions, 0 deletions