From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH v2 2/6] config: add clang support for armv8a linuxapp Date: Fri, 12 May 2017 10:47:03 +0530 Message-ID: <20170512051701.GA6557@jerin> References: <20170510101643.30556-2-ashwin.sekhar@caviumnetworks.com> <20170511143316.45835-1-ashwin.sekhar@caviumnetworks.com> <20170511143316.45835-3-ashwin.sekhar@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: thomas@monjalon.net, maciej.czekaj@caviumnetworks.com, viktorin@rehivetech.com, jianbo.liu@linaro.org, bruce.richardson@intel.com, pablo.de.lara.guarch@intel.com, konstantin.ananyev@intel.com, dev@dpdk.org To: Ashwin Sekhar T K Return-path: Received: from NAM02-SN1-obe.outbound.protection.outlook.com (mail-sn1nam02on0043.outbound.protection.outlook.com [104.47.36.43]) by dpdk.org (Postfix) with ESMTP id 67D1D2952 for ; Fri, 12 May 2017 07:17:21 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20170511143316.45835-3-ashwin.sekhar@caviumnetworks.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" -----Original Message----- > Date: Thu, 11 May 2017 07:33:12 -0700 > From: Ashwin Sekhar T K > To: jerin.jacob@caviumnetworks.com, thomas@monjalon.net, > maciej.czekaj@caviumnetworks.com, viktorin@rehivetech.com, > jianbo.liu@linaro.org, bruce.richardson@intel.com, > pablo.de.lara.guarch@intel.com, konstantin.ananyev@intel.com > Cc: dev@dpdk.org, Ashwin Sekhar T K > Subject: [dpdk-dev] [PATCH v2 2/6] config: add clang support for armv8a > linuxapp > X-Mailer: git-send-email 2.12.2 > > Moved all common defines from defconfig_arm64-armv8a-linuxapp-gcc > to common_armv8a_linuxapp. > > Created new config arm64-armv8a-linuxapp-clang which adds the > clang support to armv8a. > > Now defconfigs arm64-armv8a-linuxapp-gcc/clang contain only the > CONFIG_RTE_TOOLCHAIN* defines and all other common defines are > inherited from common_armv8a_linuxapp. > > Signed-off-by: Ashwin Sekhar T K You can move this patch at the end of the series(i.e at 6/6) so that when you introduce the clang config, it will build successfully.At 2/6 clang wont build. With that change: Reviewed-by: Jerin Jacob