[llvm] r316064 - Fix the incorrect detection of ICONV_LIBRARY_PATH
Serguei Katkov via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 17 23:26:40 PDT 2017
Author: skatkov
Date: Tue Oct 17 23:26:39 2017
New Revision: 316064
URL: http://llvm.org/viewvc/llvm-project?rev=316064&view=rev
Log:
Fix the incorrect detection of ICONV_LIBRARY_PATH
This is introduced in rL308711.
Check for c library is incorrect here just because libc will be found always
and it does not mean that iconv is presented.
Thank to Andrew Krasny for narrowing down the root cause.
Reviewers: ecbeckmann
Reviewed By: ecbeckmann
Subscribers: mgorny, llvm-commits
Differential Revision: https://reviews.llvm.org/D38875
Modified:
llvm/trunk/cmake/config-ix.cmake
Modified: llvm/trunk/cmake/config-ix.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/config-ix.cmake?rev=316064&r1=316063&r2=316064&view=diff
==============================================================================
--- llvm/trunk/cmake/config-ix.cmake (original)
+++ llvm/trunk/cmake/config-ix.cmake Tue Oct 17 23:26:39 2017
@@ -157,7 +157,7 @@ if( NOT PURE_WINDOWS AND NOT LLVM_USE_SA
set(HAVE_TERMINFO 0)
endif()
- find_library(ICONV_LIBRARY_PATH NAMES iconv libiconv libiconv-2 c)
+ find_library(ICONV_LIBRARY_PATH NAMES iconv libiconv libiconv-2)
set(LLVM_LIBXML2_ENABLED 0)
set(LIBXML2_FOUND 0)
if((LLVM_ENABLE_LIBXML2) AND ((CMAKE_SYSTEM_NAME MATCHES "Linux") AND (ICONV_LIBRARY_PATH) OR APPLE))
More information about the llvm-commits
mailing list