vitalybuka added a comment. In D99381#2721619 <https://reviews.llvm.org/D99381#2721619>, @phosek wrote:
> What I think @vitalybuka meant is keeping > `sanitizer_platform_limits_fuchsia.h` as you had it, but including it > unconditionally. Since the entire file is wrapped in `#if SANITIZER_FUCHSIA > ... #endif`, the inclusion would be a no-op on platforms other than Fuchsia > so no need to wrap the `#include` in `#if SANITIZER_FUCHSIA` as well. Correct., Just: #include "sanitizer_common/sanitizer_platform_limits_fuchsia.h" #include "sanitizer_common/sanitizer_platform_limits_posix.h" Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99381/new/ https://reviews.llvm.org/D99381 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
