[llvm] 80ed7f8 - [cmake] Remove confusing condition argument from else() NFC

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 13 16:37:18 PST 2019


Author: Tom Stellard
Date: 2019-11-13T16:37:04-08:00
New Revision: 80ed7f8397cebd2acc09a062069ae9185eed68d4

URL: https://github.com/llvm/llvm-project/commit/80ed7f8397cebd2acc09a062069ae9185eed68d4
DIFF: https://github.com/llvm/llvm-project/commit/80ed7f8397cebd2acc09a062069ae9185eed68d4.diff

LOG: [cmake] Remove confusing condition argument from else() NFC

Summary:
This makes it look like an elseif and also the variable referenced
in the condition was removed from this function in r366622.

Reviewers: dsanders, beanz, smeenai, compnerd, phosek

Reviewed By: beanz

Subscribers: mgorny, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D70159

Added: 
    

Modified: 
    llvm/cmake/modules/LLVM-Config.cmake

Removed: 
    


################################################################################
diff  --git a/llvm/cmake/modules/LLVM-Config.cmake b/llvm/cmake/modules/LLVM-Config.cmake
index 823c79d667c7..49dbac6b466a 100644
--- a/llvm/cmake/modules/LLVM-Config.cmake
+++ b/llvm/cmake/modules/LLVM-Config.cmake
@@ -268,7 +268,7 @@ function(llvm_map_components_to_libnames out_libs)
       # already processed
     elseif( c STREQUAL "all" )
       list(APPEND expanded_components ${LLVM_AVAILABLE_LIBS})
-    else( NOT idx LESS 0 )
+    else()
       # Canonize the component name:
       string(TOUPPER "${c}" capitalized)
       list(FIND capitalized_libs LLVM${capitalized} lib_idx)


        


More information about the llvm-commits mailing list