[PATCH] D99381: [compiler-rt][hwasan] Add Fuchsia-specific sanitizer platform limits

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 27 19:41:31 PDT 2021


phosek added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99381



More information about the cfe-commits mailing list