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

Kai Luo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 02:30:52 PDT 2021


lkail added a comment.

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 <lauro.venancio at gmail.com>
  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.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106844/new/

https://reviews.llvm.org/D106844



More information about the llvm-commits mailing list