I’d be amenable to that. @Sean<mailto:sean.bro...@microsoft.com>?

- Bret

From: Leif Lindholm via groups.io<mailto:leif=nuviainc....@groups.io>
Sent: Tuesday, November 2, 2021 2:51 AM
To: brbar...@microsoft.com<mailto:b...@corthon.com>
Cc: devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Ard 
Biesheuvel<mailto:ardb+tianoc...@kernel.org>; Kinney, Michael 
D<mailto:michael.d.kin...@intel.com>; Liming 
Gao<mailto:gaolim...@byosoft.com.cn>; Zhiguang 
Liu<mailto:zhiguang....@intel.com>; Sami Mujawar<mailto:sami.muja...@arm.com>; 
Yao, Jiewen<mailto:jiewen....@intel.com>; Supreeth 
Venkatesh<mailto:supreeth.venkat...@arm.com>; Maciej 
Rabeda<mailto:maciej.rab...@linux.intel.com>; Jiaxin 
Wu<mailto:jiaxin...@intel.com>; Siyuan Fu<mailto:siyuan...@intel.com>; Ni, 
Ray<mailto:ray...@intel.com>; Zhichao Gao<mailto:zhichao....@intel.com>; Sean 
Brogan<mailto:sean.bro...@microsoft.com>
Subject: [EXTERNAL] 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://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugzilla.tianocore.org%2Fshow_bug.cgi%3Fid%3D3649&amp;data=04%7C01%7CBret.Barkelew%40microsoft.com%7C6acadbd5a03f44edf9c408d99de642ee%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637714434603817528%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=AmwBKl6uWTs1VYo3Hw3I3RglkhPrWXhKPkobrwQid84%3D&amp;reserved=0
>

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 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 (#83153): https://edk2.groups.io/g/devel/message/83153
Mute This Topic: https://groups.io/mt/86775234/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to