[llvm] r315556 - [cfi-verify] Fix unittest failures w/o x86 target

Maxim Kuvyrkov via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 12 06:08:57 PDT 2017


> On Oct 12, 2017, at 7:17 AM, Vlad Tsyrklevich via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> Author: vlad.tsyrklevich
> Date: Wed Oct 11 21:17:33 2017
> New Revision: 315556
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=315556&view=rev
> Log:
> [cfi-verify] Fix unittest failures w/o x86 target
> 
> The llvm-cfi-verify unit tests fail if LLVM is built without the X86
> target, disable the unit tests from being built unless X86 is enabled
> for now.

Still broken, unfortunately.

> 
> Modified:
>    llvm/trunk/unittests/tools/llvm-cfi-verify/CMakeLists.txt
> 
> Modified: llvm/trunk/unittests/tools/llvm-cfi-verify/CMakeLists.txt
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/tools/llvm-cfi-verify/CMakeLists.txt?rev=315556&r1=315555&r2=315556&view=diff
> ==============================================================================
> --- llvm/trunk/unittests/tools/llvm-cfi-verify/CMakeLists.txt (original)
> +++ llvm/trunk/unittests/tools/llvm-cfi-verify/CMakeLists.txt Wed Oct 11 21:17:33 2017
> @@ -11,5 +11,8 @@ set(LLVM_LINK_COMPONENTS
>   Support
>   )
> 
> -add_llvm_unittest(CFIVerifyTests
> -  FileAnalysis.cpp)
> +list(FIND LLVM_ARGETS_TO_BUILD "X86" x86_idx)

s/_ARGETS/_TARGETS/

> +if (NOT x86_idx LESS 0)
> +  add_llvm_unittest(CFIVerifyTests
> +    FileAnalysis.cpp)
> +endif()

Regards,

--
Maxim Kuvyrkov
www.linaro.org




More information about the llvm-commits mailing list