[PATCH] D38911: [cfi] Test cross-dso CFI on Android.

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 13 16:52:19 PDT 2017


eugenis added inline comments.


================
Comment at: compiler-rt/test/lit.common.cfg:214
+    lit_config.fatal("Failed to read ro.build.version.sdk (using '%s' as adb): got '%s'" % (adb, android_api_level_str))
+  for x in range(14, android_api_level + 1):
+      config.available_features.add('android-' + str(x));
----------------
This could be a bit excessive...
It would add ~13 new features, and we currently only use one (android-26). AFAIK, lit does not allow numeric comparisons in UNSUPPORTED and REQUIRES headers.


https://reviews.llvm.org/D38911





More information about the llvm-commits mailing list