[PATCH] D139752: cmake: Enable 64bit off_t on 32bit glibc systems

Alex Reinking via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 6 11:17:01 PST 2023


alexreinking added a comment.

> Somehow, there is a bad interaction between the space/semicolon mixing and CMake's argument de-duplication.

Actually, upon closer inspection, it has to do with the presence of an `=` in the list. When there's an `=`, it seems that CMake does not split on spaces automatically anymore. We've never built Halide on AIX or SunOS, which are the only other build scenarios where an `=` can appear in `LLVM_DEFINITIONS`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139752



More information about the llvm-commits mailing list