From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail5.wrs.com (mail5.windriver.com [192.103.53.11]) by mail.openembedded.org (Postfix) with ESMTP id 320C975E2D for ; Fri, 6 Nov 2015 03:01:46 +0000 (UTC) Received: from ALA-HCA.corp.ad.wrs.com (ala-hca.corp.ad.wrs.com [147.11.189.40]) by mail5.wrs.com (8.15.2/8.15.2) with ESMTPS id tA631iIY027883 (version=TLSv1 cipher=AES128-SHA bits=128 verify=OK); Thu, 5 Nov 2015 19:01:44 -0800 Received: from [128.224.162.186] (128.224.162.186) by ALA-HCA.corp.ad.wrs.com (147.11.189.40) with Microsoft SMTP Server id 14.3.248.2; Thu, 5 Nov 2015 19:01:43 -0800 To: Mark Hatle , References: <88477fa50093b85e65b17c746a637c7259b87018.1446711613.git.liezhi.yang@windriver.com> <563B5D05.9060609@windriver.com> From: Robert Yang Message-ID: <563C1815.3060703@windriver.com> Date: Fri, 6 Nov 2015 11:01:41 +0800 User-Agent: Mozilla/5.0 (X11; Linux i686; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <563B5D05.9060609@windriver.com> Cc: Richard Purdie Subject: Re: [PATCH 1/1] gcc-multilib-config: make aarch64 support multilib X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 06 Nov 2015 03:01:47 -0000 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit On 11/05/2015 09:43 PM, Mark Hatle wrote: > On 11/5/15 2:20 AM, Robert Yang wrote: >> Fixed: >> MACHINE = qemuarm64 >> require conf/multilib.conf >> MULTILIBS = "multilib:lib32" >> DEFAULTTUNE_virtclass-multilib-lib32 = "armv7at-neon" >> >> $ bitbake core-image-minimal -cpopulate_sdk >> >> WARNING: gcc multilib setup is not supported for TARGET_ARCH=aarch64 >> WARNING: gcc multilib setup is not supported for TARGET_ARCH=aarch64 >> >> [YOCTO #8638] >> >> Signed-off-by: Robert Yang >> --- >> meta/recipes-devtools/gcc/gcc-multilib-config.inc | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/meta/recipes-devtools/gcc/gcc-multilib-config.inc b/meta/recipes-devtools/gcc/gcc-multilib-config.inc >> index 1c0a45a..a0a2ac0 100644 >> --- a/meta/recipes-devtools/gcc/gcc-multilib-config.inc >> +++ b/meta/recipes-devtools/gcc/gcc-multilib-config.inc >> @@ -29,6 +29,9 @@ python gcc_multilib_setup() { >> bb.utils.remove(build_conf_dir, True) >> ml_globs = ('%s/*/t-linux64' % src_conf_dir, >> '%s/*/linux64.h' % src_conf_dir, >> + '%s/aarch64/t-aarch64' % src_conf_dir, >> + '%s/aarch64/aarch64.h' % src_conf_dir, >> + '%s/aarch64/aarch64-cores.def' % src_conf_dir, >> '%s/*/linux.h' % src_conf_dir, >> '%s/linux.h' % src_conf_dir) >> >> @@ -130,6 +133,8 @@ python gcc_multilib_setup() { >> 'mips64' : ['gcc/config/mips/t-linux64'], >> 'powerpc' : ['gcc/config/rs6000/t-linux64'], >> 'powerpc64' : ['gcc/config/rs6000/t-linux64'], >> + 'aarch64' : ['gcc/config/aarch64/t-aarch64'], >> + 'arm' : ['gcc/config/aarch64/t-aarch64'], >> } >> >> gcc_header_config_files = { >> @@ -140,6 +145,8 @@ python gcc_multilib_setup() { >> 'mips64' : ['gcc/config/mips/linux.h', 'gcc/config/mips/linux64.h'], >> 'powerpc' : ['gcc/config/rs6000/linux64.h'], >> 'powerpc64' : ['gcc/config/rs6000/linux64.h'], >> + 'aarch64' : ['gcc/config/aarch64/aarch64.h'], >> + 'arm' : ['gcc/config/aarch64/aarch64.h'], >> } > > I'm not sure the above is correct in this case. As I believe GCC treats aarch64 > and arm as different architectures unlike MIPS, Power and IA32. > > In this case, I would expect two specific cross compilers to be generated, one > for armv7 and one for aarch64, instead of a combination single compiler that > understand both. Hi Mark, I think that there are two compilers ? For example, gcc and lib32-gcc. Yes, Hopefully we can get feed back from Khem or RP. // Robert > > So in this case, it might be better to have an exception to the warning > message... (unless I'm wrong, in which case this IS likely a correct fix....) > > Hopefully Khem and/or Richard will be able to comment as they are more familiar > with this code path. > >> >> libdir32 = 'SYSTEMLIBS_DIR' >> > >