Snap for 5681426 from 09a8725d56b168668a11530537c9738f4bc58a90 to qt-releaseandroid-vts-10.0_r1android-cts-10.0_r1android-10.0.0_r6android-10.0.0_r5android-10.0.0_r46android-10.0.0_r4android-10.0.0_r3android-10.0.0_r2android-10.0.0_r17android-10.0.0_r11android-10.0.0_r10android-10.0.0_r1android10-security-releaseandroid10-s3-releaseandroid10-s2-releaseandroid10-s1-releaseandroid10-release

Change-Id: I56762c1b6ea38c57c077c2fbdb6fb4a6231f4292

2 files changed, 78 insertions, 22 deletions

diff --git a/Android.bp b/Android.bp
index f372618..a5901e0 100644
--- a/Android.bp
+++ b/Android.bp
@@ -18,27 +18,10 @@ subdirs = [
 
 cc_defaults {
     name: "libhwbinder_defaults",
-    shared_libs: [
-        "libbase",
-        "liblog",
-        "libcutils",
-        "libutils",
-        "libbinderthreadstate",
-    ],
-    export_shared_lib_headers: [
-        "libbase",
-        "libutils",
-    ],
+
     export_include_dirs: ["include"],
     include_dirs: ["frameworks/native/include"],
 
-    recovery_available: true,
-    vendor_available: true,
-    vndk: {
-        enabled: true,
-        support_system_process: true,
-    },
-    clang: true,
     sanitize: {
         misc_undefined: ["integer"],
     },
@@ -67,22 +50,87 @@ cc_defaults {
     ],
 }
 
+cc_defaults {
+    name: "libhwbinder-impl-shared-libs",
+    defaults: ["libhwbinder-impl-shared-libs-no-vndk-private"],
+    shared_libs: [
+        "libbinderthreadstate",
+    ],
+}
+
+cc_defaults {
+    name: "libhwbinder-impl-shared-libs-no-vndk-private",
+    shared_libs: [
+        "libbase",
+        "liblog",
+        "libcutils",
+        "libutils",
+    ],
+    export_shared_lib_headers: [
+        "libbase",
+        "libutils",
+    ],
+}
+
+// WARNING: this should no longer be used
 cc_library {
     name: "libhwbinder",
+    recovery_available: true,
+    vendor_available: true,
+    vndk: {
+        enabled: true,
+        support_system_process: true,
+    },
+
+    export_include_dirs: ["include"],
+}
+
+// Combined into libhidlbase for efficiency.
+// Used as shared library to provide headers for libhidltransport-impl-internal.
+cc_library_static {
+    name: "libhwbinder-impl-internal",
+    // TODO(b/135299443): allow this library to link against vndk-private libs
+    // and instead rely on the fact that users of this static library must be
+    // vndk (since they must use libbinderthreadstate).
+    include_dirs: ["frameworks/native/libs/binderthreadstate/include/"],
     defaults: [
         "libhwbinder_defaults",
+        "libhwbinder-impl-shared-libs-no-vndk-private",
         "hwbinder_pgo",
         "hwbinder_lto",
     ],
+    recovery_available: true,
+    vendor_available: true,
 }
 
 // Explicitly provide a no lto, no PGO variant, to workaround the issue that we
 // can't detect non-lto users of the module in Android.mk.
-// http://b/77320844
+// TODO(b/135558503): remove
 cc_library {
     name: "libhwbinder_noltopgo",
     defaults: [
         "libhwbinder_defaults",
+        "libhwbinder-impl-shared-libs",
+    ],
+    recovery_available: true,
+    vendor_available: true,
+    vndk: {
+        enabled: true,
+        support_system_process: true,
+    },
+}
+
+// Only libhwbinder_benchmark needs to have pgo enabled. When all places
+// support having PGO selectively enabled, all places can use libhwbinder.
+//
+// http://b/77320844
+cc_library_static {
+    name: "libhwbinder_pgo-impl-internal",
+    defaults: [
+        "libhwbinder_defaults",
+        "libhwbinder-impl-shared-libs",
+        "hwbinder_benchmark_pgo",
+        "hwbinder_lto",
     ],
 }
 
@@ -97,6 +145,16 @@ cc_defaults {
     },
 }
 
+cc_defaults {
+    name: "hwbinder_benchmark_pgo",
+    pgo: {
+        instrumentation: true,
+        profile_file: "hwbinder/hwbinder.profdata",
+        benchmarks: ["hwbinder_benchmark"],
+        enable_profile_use: true,
+    },
+}
+
 // Provide lto property to build hwbinder with LTO
 cc_defaults {
     name: "hwbinder_lto",
diff --git a/vts/performance/Android.bp b/vts/performance/Android.bp
index b5097df..38edce5 100644
--- a/vts/performance/Android.bp
+++ b/vts/performance/Android.bp
@@ -22,9 +22,7 @@ cc_defaults {
         "-Werror",
     ],
     shared_libs: [
-        "libhidlbase",
-        "libhidltransport",
-        "libhwbinder",
+        "libhidlbase_pgo",
         "liblog",
         "libutils",
         "libcutils",