> On Nov 3, 2021, at 11:23 PM, gaoliming <gaolim...@byosoft.com.cn> wrote:
>
> 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.
>
+1
Thanks,
Andrew Fish
> 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 (#83321): https://edk2.groups.io/g/devel/message/83321
Mute This Topic: https://groups.io/mt/86810475/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
