[Lldb-commits] [lldb] [lldb] Include api-level.h header for Android (PR #124383)

via lldb-commits lldb-commits at lists.llvm.org
Mon Jan 27 06:51:00 PST 2025


================
@@ -29,6 +29,10 @@
 #include <sys/utsname.h>
 #include <unistd.h>
 
+#ifdef __ANDROID__
+#include <android/api-level.h>
----------------
enh-google wrote:

what's the build failure you're seeing? this _should_ be included "for free" by all of the #includes above (via <sys/cdefs.h>, which they should all drag in).

of course, if you're aiming for a style guide "explicitly include what you use" kind of thing, that's fine, but -- other than for style reasons -- you shouldn't ever _need_ this...

https://github.com/llvm/llvm-project/pull/124383


More information about the lldb-commits mailing list