From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2783CA0C4E; Thu, 22 Jul 2021 09:47:03 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1252F4014E; Thu, 22 Jul 2021 09:47:03 +0200 (CEST) Received: from shelob.oktetlabs.ru (shelob.oktetlabs.ru [91.220.146.113]) by mails.dpdk.org (Postfix) with ESMTP id 66F4C4014D for ; Thu, 22 Jul 2021 09:47:01 +0200 (CEST) Received: by shelob.oktetlabs.ru (Postfix, from userid 122) id D5D887F596; Thu, 22 Jul 2021 10:47:00 +0300 (MSK) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on shelob.oktetlabs.ru X-Spam-Level: X-Spam-Status: No, score=0.8 required=5.0 tests=ALL_TRUSTED, DKIM_ADSP_DISCARD, URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.2 Received: from aros.oktetlabs.ru (aros.oktetlabs.ru [192.168.38.17]) by shelob.oktetlabs.ru (Postfix) with ESMTP id 481437F535; Thu, 22 Jul 2021 10:46:57 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 shelob.oktetlabs.ru 481437F535 Authentication-Results: shelob.oktetlabs.ru/481437F535; dkim=none; dkim-atps=neutral From: Andrew Rybchenko To: dev@dpdk.org Cc: alialnu@nvidia.com, David Marchand , Bruce Richardson Date: Thu, 22 Jul 2021 10:46:51 +0300 Message-Id: <20210722074651.1837621-1-andrew.rybchenko@oktetlabs.ru> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH] net/sfc: fix broken build with clang 3.4.x X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Old clanng requires libatomic as well as gcc. Avoid compiler name and version based checks. Add custom test for 16-byte atomic operations to find out if libatomic is required to build. Bugzilla ID: 760 Signed-off-by: Andrew Rybchenko --- drivers/net/sfc/meson.build | 22 +++++++++++++++------- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/drivers/net/sfc/meson.build b/drivers/net/sfc/meson.build index 4625859077..a1ad792b80 100644 --- a/drivers/net/sfc/meson.build +++ b/drivers/net/sfc/meson.build @@ -40,8 +40,20 @@ foreach flag: extra_flags endif endforeach -# for gcc compiles we need -latomic for 128-bit atomic ops -if cc.get_id() == 'gcc' +# for gcc and old Clang compiles we need -latomic for 128-bit atomic ops +atomic_check_code = ''' +int main(void) +{ + __int128 a = 0; + __int128 b; + + b = __atomic_load_n(&a, __ATOMIC_RELAXED); + __atomic_store(&b, &a, __ATOMIC_RELAXED); + __atomic_store_n(&b, a, __ATOMIC_RELAXED); + return 0; +} +''' +if not cc.links(atomic_check_code) libatomic_dep = cc.find_library('atomic', required: false) if not libatomic_dep.found() build = false @@ -51,11 +63,7 @@ if cc.get_id() == 'gcc' # libatomic could be half-installed when above check finds it but # linkage fails - atomic_link_code = ''' - #include - void main() { printf("libatomic link check\n"); } - ''' - if not cc.links(atomic_link_code, dependencies: libatomic_dep) + if not cc.links(atomic_check_code, dependencies: libatomic_dep) build = false reason = 'broken dependency, "libatomic"' subdir_done() -- 2.30.2