From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH v2] eal: fix clang compilation error on ARM64 Date: Wed, 11 Apr 2018 14:48:24 -0700 Message-ID: <20180411144824.062e1432@xeon-e3> References: <20180406110103.29163-1-pbhagavatula@caviumnetworks.com> <20180411170150.11985-1-pbhagavatula@caviumnetworks.com> <2796123.plAjDWPncV@xps> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Pavan Nikhilesh , dev@dpdk.org, jerin.jacob@caviumnetworks.com, ferruh.yigit@intel.com, konstantin.ananyev@intel.com To: Thomas Monjalon Return-path: Received: from mail-pf0-f193.google.com (mail-pf0-f193.google.com [209.85.192.193]) by dpdk.org (Postfix) with ESMTP id 687091B73F for ; Wed, 11 Apr 2018 23:48:27 +0200 (CEST) Received: by mail-pf0-f193.google.com with SMTP id y69so1790160pfb.5 for ; Wed, 11 Apr 2018 14:48:27 -0700 (PDT) In-Reply-To: <2796123.plAjDWPncV@xps> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, 11 Apr 2018 22:42:55 +0200 Thomas Monjalon wrote: > 11/04/2018 19:01, Pavan Nikhilesh: > > Use __atomic_exchange_n instead of __atomic_exchange_(2/4/8). > > The error was: > > include/generic/rte_atomic.h:215:9: error: > > implicit declaration of function '__atomic_exchange_2' > > is invalid in C99 > > include/generic/rte_atomic.h:494:9: error: > > implicit declaration of function '__atomic_exchange_4' > > is invalid in C99 > > include/generic/rte_atomic.h:772:9: error: > > implicit declaration of function '__atomic_exchange_8' > > is invalid in C99 > > > > Fixes: ff2863570fcc ("eal: introduce atomic exchange operation") > > > > Signed-off-by: Pavan Nikhilesh > > We did not understand why this error is seen only on ARM > (hopefully we won't discover it somewhere else), > but it is a pragmatic fix. > > Applied, thanks Maybe Gcc always defines its own builtin functions.