Lefi:

> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Leif Lindholm
> 发送时间: 2021年11月2日 17:51
> 收件人: brbar...@microsoft.com <b...@corthon.com>
> 抄送: devel@edk2.groups.io; Ard Biesheuvel <ardb+tianoc...@kernel.org>;
> Michael D Kinney <michael.d.kin...@intel.com>; Liming Gao
> <gaolim...@byosoft.com.cn>; Zhiguang Liu <zhiguang....@intel.com>; Sami
> Mujawar <sami.muja...@arm.com>; Jiewen Yao <jiewen....@intel.com>;
> Supreeth Venkatesh <supreeth.venkat...@arm.com>; Maciej Rabeda
> <maciej.rab...@linux.intel.com>; Jiaxin Wu <jiaxin...@intel.com>; Siyuan
> Fu <siyuan...@intel.com>; Ray Ni <ray...@intel.com>; Zhichao Gao
> <zhichao....@intel.com>; Sean Brogan <sean.bro...@microsoft.com>
> 主题: Re: [edk2-devel] [PATCH v1 05/16] ArmPkg and MdePkg: Move the Arm
> CompilerIntrinsicsLib to MdePkg
> 
> On Mon, Nov 01, 2021 at 12:56:37 -0700, brbar...@microsoft.com wrote:
> > From: Bret Barkelew <brbar...@microsoft.com>
> >
> > This aligns better with Mu's philosophy around dependency structuring
> > and is one of the steps to enable Basecore to have zero CI dependencies
> > on other Mu repos.
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3649
> >
> 
> Only one comment on this patch really.
> 
> >  ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf =>
> MdePkg/Library/CompilerIntrinsicsLib/ArmCompilerIntrinsicsLib.inf | 3 +--
> 
> Could we just keep the .inf name as is?
> 
I agree your suggestion. CompilerIntrinsicsLib can support X86 in future.
So, 
CompilerIntrinsicsLib.inf should be used. 

Thanks
Liming

> I think we're getting closer to the x86 folk accepting that they
> need this too, rather than keep inventing new dialects of C in the
> desperate hope that the compiler won't generate stdlib calls it's
> fully permitted to generate whenever it feels like.
> 
> /
>     Leif
> 
> 
> 
> 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83320): https://edk2.groups.io/g/devel/message/83320
Mute This Topic: https://groups.io/mt/86810473/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to