Fix CodeBuffer implementation mismatch.android-n-iot-release-lg-thinq-wk7

Test: Rely on TreeHugger.
Bug: 64359573
Change-Id: I17845fea4031032de1dc200494a57e91fbaa2de0

1 files changed, 8 insertions, 2 deletions

diff --git a/Android.bp b/Android.bp
index d2d73906..d10bad34 100644
--- a/Android.bp
+++ b/Android.bp
@@ -72,6 +72,14 @@ cc_defaults {
         // Explicitly enable the write-strings warning. VIXL uses
         // const correctly when handling string constants.
         "-Wwrite-strings",
+
+        // As we can load both libvixl-arm.so and libvixl-arm64.so in the
+        // same process, and some functions are emitted in both libraries,
+        // flags for selecting implementation details for these functions
+        // must be shared to emit identical code. Otherwise we can get
+        // undefined behavior, especially with link-time optimization.
+        // b/64359573
+        "-DVIXL_CODE_BUFFER_MALLOC",
     ],
     local_include_dirs: ["src"],
     native_coverage: false,
@@ -84,7 +92,6 @@ cc_defaults {
     name: "vixl-arm",
     defaults: ["vixl-common"],
     cppflags: [
-        "-DVIXL_CODE_BUFFER_MALLOC",
         "-DVIXL_INCLUDE_TARGET_T32",
         ],
     srcs: ["src/aarch32/*.cc"],
@@ -95,7 +102,6 @@ cc_defaults {
     defaults: ["vixl-common"],
     cppflags: [
         "-DVIXL_INCLUDE_SIMULATOR_AARCH64",
-        "-DVIXL_CODE_BUFFER_MMAP",
         "-DVIXL_INCLUDE_TARGET_A64",
         ],
     srcs: ["src/aarch64/*.cc"],