[PATCH] D55953: Android is not GNU, so don't claim that it is.

Kristina Brooks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 30 03:07:20 PST 2018


kristina added a comment.

Seems good, it could eliminate the need for a lot of preprocessor checks like `#if __gnu_linux__ && !defined(__ANDROID__)`. It doesn't seem that there are any preprocessor checks where this would cause problems (from a quick search) since all of them seem to focus on making sure `__gnu_linux__` being defined explicitly excludes Android from those paths.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55953





More information about the cfe-commits mailing list