Skip to content

Commit

Permalink
llvm-isel-fuzzer: Work around BUILD_SHARED_LIBS testing issues
Browse files Browse the repository at this point in the history
Building with BUILD_SHARED_LIBS makes it tricky to copy around
executables at will, since they won't be able to find the LLVM
libraries any more. This makes testing a feature that's based on the
executable name problematic, so we'll just disable these two tests in
that configuration.

We could potentially fix this by symlinking the lib directory into the
test directory, but that wouldn't work on windows, and losing testing
on windows would be far worse than losing testing on a configuration
that's barely even supported.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@315599 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
bogner committed Oct 12, 2017
1 parent 3df0d5d commit c209885
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 0 deletions.
4 changes: 4 additions & 0 deletions test/tools/llvm-isel-fuzzer/aarch64-execname-options.ll
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
; If the binary looks up libraries using an rpath, we can't test this
; without copying the whole lib dir or polluting the build dir.
; UNSUPPORTED: rpath_in_use

; REQUIRES: aarch64-registered-target

; RUN: echo > %t.input
Expand Down
4 changes: 4 additions & 0 deletions test/tools/llvm-isel-fuzzer/execname-options.ll
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
; If the binary looks up libraries using an rpath, we can't test this
; without copying the whole lib dir or polluting the build dir.
; UNSUPPORTED: rpath_in_use

; RUN: echo > %t.input

; RUN: cp llvm-isel-fuzzer %t.bin=gisel
Expand Down
2 changes: 2 additions & 0 deletions test/tools/llvm-isel-fuzzer/lit.local.cfg
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
if config.enable_shared == 1:
config.available_features.add('rpath_in_use')

0 comments on commit c209885

Please sign in to comment.