From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web10.4783.1647566844573706143 for ; Thu, 17 Mar 2022 18:27:25 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([101.224.116.119]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Fri, 18 Mar 2022 09:26:43 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 101.224.116.119 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: "'Gerd Hoffmann'" , , Cc: "'Sean Brogan'" , "'Lu, Xiaoyu1'" , "'Feng, Bob C'" , "'Rebecca Cran'" , "'James Bottomley'" , "'Sami Mujawar'" , "'Justen, Jordan L'" , "'Aktas, Erdem'" , "'Yao, Jiewen'" , "'Supreeth Venkatesh'" , "'Boeuf, Sebastien'" , "'Gao, Zhichao'" , "'Liu, Zhiguang'" , "'Maciej Rabeda'" , "'Ma, Maurice'" , "'Andrew Fish'" , "'Ard Biesheuvel'" , "'Tom Lendacky'" , "'Peter Grehan'" , "'Jiang, Guomin'" , "'Bret Barkelew'" , "'Chen, Christine'" , "'You, Benjamin'" , "'Schaefer, Daniel'" , "'Xu, Wei6'" , "'Wang, Jian J'" , "'Wu, Jiaxin'" , "'Fu, Siyuan'" , "'Dong, Guo'" , , "'Chang, Abner'" , "'Oliver Steffen'" , "'Leif Lindholm'" , "'Brijesh Singh'" , "'Xu, Min M'" , "'Ni, Ray'" , "'Alexei Fedorov'" , "'Julien Grall'" , "'Wang, Nickle'" , "'Pawel Polawski'" , "'Anthony Perard'" References: <20220301071820.3135133-1-kraxel@redhat.com> <20220315122242.xwi2xckfos2c3cff@sirius.home.kraxel.org> <20220316094538.t4ox4p5amfgwt5t6@sirius.home.kraxel.org> In-Reply-To: <20220316094538.t4ox4p5amfgwt5t6@sirius.home.kraxel.org> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIDAvM10gW1JGQ10gY29uc29saWRhdGUgY29tcGlsZXIgaW50cmluc2ljcw==?= Date: Fri, 18 Mar 2022 09:26:45 +0800 Message-ID: <018801d83a67$3c3ef0f0$b4bcd2d0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQHicnmkzsTVFiDsYWAVCsx9jXmBNAHs5agtApkIOwkCFiUGrKx7CBmQ Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Gerd: I suggest to submit the different BZs for those tasks. This patch set = is the first stage. Future enhancement can be added later. If no more comment for the first stage, this patch set can be merged first. Thanks Liming > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: Gerd Hoffmann > =B7=A2=CB=CD=CA=B1=BC=E4: 2022=C4=EA3=D4=C216=C8=D5 17:46 > =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io; michael.d.kinney@intel.com > =B3=AD=CB=CD: Sean Brogan ; Lu, Xiaoyu1 > ; Feng, Bob C ; Rebecca = Cran > ; James Bottomley ; Sami > Mujawar ; Justen, Jordan L > ; Aktas, Erdem ; = Yao, > Jiewen ; Supreeth Venkatesh > ; Boeuf, Sebastien > ; Gao, Zhichao ; = Liu, > Zhiguang ; Maciej Rabeda > ; Ma, Maurice ; > Andrew Fish ; Ard Biesheuvel > ; Tom Lendacky ; > Peter Grehan ; Jiang, Guomin > ; Bret Barkelew ; > Chen, Christine ; You, Benjamin > ; Schaefer, Daniel ; > Gao, Liming ; Xu, Wei6 ; > Wang, Jian J ; Wu, Jiaxin = ; Fu, > Siyuan ; Dong, Guo ; > kilian_kegel@hotmail.com; Chang, Abner ; Oliver > Steffen ; Leif Lindholm = ; > Brijesh Singh ; Xu, Min M ; > Ni, Ray ; Alexei Fedorov ; > Julien Grall ; Wang, Nickle ; = Pawel > Polawski ; Anthony Perard > > =D6=F7=CC=E2: Re: [edk2-devel] [PATCH 0/3] [RFC] consolidate compiler = intrinsics >=20 > On Tue, Mar 15, 2022 at 03:27:58PM +0000, Michael D Kinney wrote: > > Hi Gerd, > > > > Thanks for working on this. I raised this topic in the new = TianoCore build > and CI > > meeting yesterday because the intrinsic libs are closely tied to the > toolchains > > supported by TianoCore. > > > > I agree with the concept of moving the compiler intrinsics to a = library in the > MdePkg. > > > > + Sean Brogan > > > > Sean mentioned that he has some addition work on intrinsic libs for Visual > Studio > > toolchains. It would be good to see if this set of changes is = aligned with > those > > efforts. >=20 > Having a single place where all compiler intrinsics are located should > simplify this kind of changes. >=20 > > I see a few items that might be good to clean up at some point > > * fltused should go into its own file. Not related to the strcmp() = API > > * I see size_t defined in multiple places using different mappings. Would > be good to > > define that in a single location. >=20 > Yes, sure. Both are valid points. >=20 > I see this series as first step, consolidating the existing code in a > singe place with (almost) no actual code changes. There surely will = be > incremental improvements and cleanups later on. I will need a few = more > intrinsic functions when moving edk2 to openssl 3.0 for example. >=20 > take care, > Gerd