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

2019-01-08 Thread Dan Albert via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC350664: Android is not GNU, so don't claim that it is. (authored by danalbert, committed by ). Changed prior to commit: https://reviews.llvm.org/D55953?vs=179140&id=180741#toc Repository: rC Clang C

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

2019-01-08 Thread Stephen Hines via Phabricator via cfe-commits
srhines accepted this revision. srhines added a comment. This revision is now accepted and ready to land. Sorry about the delay. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55953/new/ https://reviews.llvm.org/D55953 __

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

2018-12-30 Thread Kristina Brooks via Phabricator via cfe-commits
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 m

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

2018-12-20 Thread Dan Albert via Phabricator via cfe-commits
danalbert created this revision. danalbert added reviewers: pirama, srhines. Repository: rC Clang https://reviews.llvm.org/D55953 Files: lib/Basic/Targets/OSTargets.h test/Preprocessor/init.c Index: test/Preprocessor/init.c