[PATCH] D46454: [sanitizer] Trivial portion of the port to Myriad RTEMS

[Walter Lee via Phabricator via llvm-commits]

waltl marked an inline comment as done.
waltl added inline comments.


================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_platform_interceptors.h:32
+#if ((SI_POSIX != 0) == (SI_WINDOWS != 0)) && !SANITIZER_FUCHSIA \
+  && !SANITIZER_RTEMS
 # error "Windows is not POSIX!"
----------------
alekseyshl wrote:
> How SANITIZER_FUCHSIA and SANITIZER_RTEMS are related to "Windows is not POSIX!" message? Both are not Windows indeed.
The check was from r309536 back when a port was either Windows or
POSIX, predating Fuchsia/RTEMS.  Even back then the error message was
a little cryptic as it would fire when a port is neither.

I can rewrite this to be slightly more clear, but I wonder whether we need
this check at all?  Vitaly can you weigh in?




Repository:
  rL LLVM

https://reviews.llvm.org/D46454