[test-suite] Fix CMakeLists.txt of MultiSource/Applications/viterbi to enable small data file

[zixuan-wu]

Summary

LLVM test-suite can reduce executing time by using small data set as input data when SMALL_PROBLEM_SIZE macro is enable.
In term of that, the CMakeLists.txt of test case should copy corresponding data set file to build dir as input data correctly by SMALL_PROBLEM_SIZE macro.

Now the viterbi case does not give appropriate data set so that it fails to run correctly in small data set condition.

[zixuan-wu]

Could anybody add some appropriate reviewers?

[lkail]

Could you elaborate more about what problem you are encountered?

[lkail]

Looks code in viterbi's test.c already checks this macro and set proper input.

#ifdef SMALL_PROBLEM_SIZE
#define INPUT_FILE PROJ_SRC_DIR "/Dist_demux_small"
#else
#define INPUT_FILE PROJ_SRC_DIR "/Dist_demux"
#endif

I think it works when it was added in

commit e7315d690fe9947fe76f43e5c3467940e51b4597
Author: Lauro Ramos Venancio <[e-mail]>
Date:   Thu May 3 18:18:17 2007 +0000

    Implement SMALL_PROBLEM_SIZE.
    
    llvm-svn: 36682

based on Makefile harness, see Makefile.rules

ifdef SMALL_PROBLEM_SIZE
CPPFLAGS += -DSMALL_PROBLEM_SIZE
endif

I think we should do the same for CMake based harness.

[Meinersbur]

llvm_test_data copies/symlinks the input data to the build dir to make running the test-suite independent of the git checkout. Indeed when using small problem size, we need to copy the small input instead of the standard size input file to the build dir. Could also just have copied both of them unconditionally.

[Meinersbur]

@zixuan-wu The summary is therefore inaccurate. llvm_test_data is not used to pass arguments to the executable, but to copy the input files. Please update the summary before committing.

[zixuan-wu]

This comment was removed by zixuan-wu.

MultiSource/Applications/viterbi/CMakeLists.txt

 list(APPEND CPPFLAGS -I${CMAKE_CURRENT_SOURCE_DIR} -DPROJ_SRC_DIR=".")
+if(SMALL_PROBLEM_SIZE)
+  set(INPUTS Dist_demux_small)
+else()
+  set(INPUTS Dist_demux)
+endif()
 llvm_multisource(viterbi)
-llvm_test_data(viterbi Dist_demux)
+llvm_test_data(viterbi ${INPUTS})