[EXTERNAL] Re: [edk2-devel] [PATCH v1 05/16] ArmPkg and MdePkg: Move the Arm CompilerIntrinsicsLib to MdePkg

[Bret Barkelew via groups.io]

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

- Bret

From: Leif Lindholm via groups.io<mailto:leif=nuviainc.com at groups.io>
Sent: Tuesday, November 2, 2021 2:51 AM
To: brbarkel at microsoft.com<mailto:bret at corthon.com>
Cc: devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Ard Biesheuvel<mailto:ardb+tianocore at kernel.org>; Kinney, Michael D<mailto:michael.d.kinney at intel.com>; Liming Gao<mailto:gaoliming at byosoft.com.cn>; Zhiguang Liu<mailto:zhiguang.liu at intel.com>; Sami Mujawar<mailto:sami.mujawar at arm.com>; Yao, Jiewen<mailto:jiewen.yao at intel.com>; Supreeth Venkatesh<mailto:supreeth.venkatesh at arm.com>; Maciej Rabeda<mailto:maciej.rabeda at linux.intel.com>; Jiaxin Wu<mailto:jiaxin.wu at intel.com>; Siyuan Fu<mailto:siyuan.fu at intel.com>; Ni, Ray<mailto:ray.ni at intel.com>; Zhichao Gao<mailto:zhichao.gao at intel.com>; Sean Brogan<mailto:sean.brogan at 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, brbarkel at microsoft.com wrote:
> From: Bret Barkelew <brbarkel at 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&data=04%7C01%7CBret.Barkelew%40microsoft.com%7C6acadbd5a03f44edf9c408d99de642ee%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637714434603817528%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=AmwBKl6uWTs1VYo3Hw3I3RglkhPrWXhKPkobrwQid84%3D&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/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20211102/75cfbeaf/attachment.htm>