[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