Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f9f03e01 authored by Ivan Lozano's avatar Ivan Lozano
Browse files

Disable AArch64 XOM when not using lld.

AArch64 execute-only memory is only supported when using lld as the
linker. There's still a few modules which don't use lld, so in those
cases we need to disable this option.

Bug: 77958880
Test: Module with LOCAL_USE_CLANG_LLD false builds without XOM
Test: Module without LOCAL_USE_CLANG_LLD defined builds with XOM

Change-Id: I5dfe3cd1e1cac2b3ead13912af9f7fe896d507e9
parent 159eb093
Loading
Loading
Loading
Loading
+3 −1
Original line number Diff line number Diff line
@@ -91,11 +91,13 @@ ifeq ($(strip $(ENABLE_XOM)),true)

    ifeq ($(strip $(my_xom)),true)
      ifeq (arm64,$(TARGET_$(LOCAL_2ND_ARCH_VAR_PREFIX)ARCH))
        ifeq ($(my_use_clang_lld),true)
          my_ldflags += -Wl,-execute-only
        endif
      endif
    endif
  endif
endif

my_allow_undefined_symbols := $(strip $(LOCAL_ALLOW_UNDEFINED_SYMBOLS))
ifdef SANITIZE_HOST