From 6d9861c494a68f76b105328db52369cccb1f1d20 Mon Sep 17 00:00:00 2001
From: Christian Kandeler <christian.kandeler@qt.io>
Date: Wed, 13 Feb 2019 10:32:01 +0100
Subject: Android: Remove "noexecstack" assembler option

It's redundant (we also specify this when linking), and it can cause
warnings with clang.

Fixes: QBS-1424
Change-Id: I71a9331ac0385c53266ef702d192b4a95d1055ba
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
---
 share/qbs/modules/Android/ndk/utils.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/share/qbs/modules/Android/ndk/utils.js b/share/qbs/modules/Android/ndk/utils.js
index a2e342897..3bc3b6d9d 100644
--- a/share/qbs/modules/Android/ndk/utils.js
+++ b/share/qbs/modules/Android/ndk/utils.js
@@ -63,7 +63,7 @@ function androidAbi(arch) {
 
 function commonCompilerFlags(toolchain, buildVariant, abi, armMode) {
     var flags = ["-ffunction-sections", "-funwind-tables",
-                 "-Wa,--noexecstack", "-Werror=format-security", "-fstack-protector-strong"];
+                 "-Werror=format-security", "-fstack-protector-strong"];
 
     if (abi === "arm64-v8a")
         flags.push("-fpic");
-- 
cgit v1.2.3