[libcxx-commits] [PATCH] D123890: [libunwind] Update the test configuration files to support the remote execution.

[Louis Dionne via Phabricator via libcxx-commits]

ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.

This looks reasonable to me, but I have a question.



================
Comment at: libunwind/test/configs/llvm-libunwind-shared.cfg.in:23
 config.test_exec_root = '@CMAKE_BINARY_DIR@'
+config.target_info = "@LIBUNWIND_TARGET_INFO@"
 
----------------
Why do you need this bit? Same for the static configuration.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123890