[PATCH] D73524: [CMake][DFSan] Don't use cat as it's not available on Windows

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 28 15:02:49 PST 2020


phosek added a comment.

In D73524#1843700 <https://reviews.llvm.org/D73524#1843700>, @tstellar wrote:

> lit has a builtin cat command, is there some way to use it here instead of re-implementing the functionality?


I don't think that LLVM build should depend on `lit`. Ideally, this would be a CMake command, which is currently not the case, but I found a way to emulate it using a combination of `file(READ ...)`, `file(APPEND ...)` and `configure_file(...)`: https://stackoverflow.com/a/15283110. @smeenai would you prefer that solution?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73524





More information about the llvm-commits mailing list