This revision was automatically updated to reflect the committed changes.
Closed by commit rL286543: Define __ANDROID_API__ when specified as part of an
Android target. (authored by srhines).
Changed prior to commit:
https://reviews.llvm.org/D26491?vs=77446&id=77578#toc
Repository:
rL LLVM
eugenis accepted this revision.
eugenis added a reviewer: eugenis.
eugenis added a comment.
Looks great!
https://reviews.llvm.org/D26491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
pirama accepted this revision.
pirama added a reviewer: pirama.
pirama added a comment.
This revision is now accepted and ready to land.
The bug points back to this Phab page instead of a bug. LGTM otherwise.
https://reviews.llvm.org/D26491
___
cfe
srhines updated this revision to Diff 77446.
srhines added a comment.
Added bug link to commit message.
https://reviews.llvm.org/D26491
Files:
lib/Basic/Targets.cpp
test/Preprocessor/init.c
Index: test/Preprocessor/init.c
===
srhines created this revision.
srhines added subscribers: danalbert, eugenis, pirama, cfe-commits.
Herald added a subscriber: tberghammer.
This macro should be defined only when the user directly specifies an
API level as part of an Android target. For any regular Android target,
we leave this mac
