From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from na01-obe.outbound.protection.outlook.com (na01-obe.outbound.protection.outlook.com [52.101.61.20]) by mx.groups.io with SMTP id smtpd.web09.388.1635882283870158006 for ; Tue, 02 Nov 2021 12:44:44 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@microsoft.com header.s=selector2 header.b=TYwVeRcO; spf=pass (domain: microsoft.com, ip: 52.101.61.20, mailfrom: bret.barkelew@microsoft.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JME+yf7MPz3DEKU3nNxaIDZJ9tgL86CuDixVMCv7XIU0hi2n87jP9psq/0840z1NSnmYtwM+vMTtPr/JC/+Nz344/HDPX7DIKGZQp+fQxensfLMWCceT3zGGa3V2bXuPhtQya7sSlf8rssT0WFmpj31ioTDM+E+7/AGiJFqZ7IUDkSC3pt5zXQFRF46lWlSCvbGhCtjf6PPqtZrTkBMCfvLhQGq57yTZtSTfLFXwtKhRys2dNrh0j5NzKRkiTiQWwJEjVlAKQ/4kq6Hqes7KU4Y9O/OZEXeFvEghDu8mz+Q81fud/PKKq4WMqYAw24+PyyPeIx59mRmjFOStpzo41A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NTMW/yutSl1FMM53ZcC7h7QvpHUKGlORppNfTOVLi7M=; b=AmduEAS2yMTLwPJQRaf83qGpr1v/gcKpYvElQ0dix3qcN64eYLMRurppi1EyKVwAYNPA1qIUw2jaMOTLzOlvReMwU8q4Rzg4/TRrJwVYC70x3b7PEsTXrJFzdPhMYP6aizc+R6MidGAGAQlNohLeg6orP38AbC57hMjyRsEKx3ALcMZUykh7pA7G5EdCnaNflkUYyuIWS9+p3FFA4QIVs3cH60EsYf2aPkPPsMTQWtS8etJqB+Z4o++fRdq27CjbbUM93YeMQ2qN1hHuuxT0ENrnNtabYMkvse5SUOMKP+6BnI3lLv781F2mK97zGUZnMuEzlGDc2+2dmz6+elSeeg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NTMW/yutSl1FMM53ZcC7h7QvpHUKGlORppNfTOVLi7M=; b=TYwVeRcOCq9cbgH22opyjl6y+DNpUqj6G9gBYZ5hesbdyiNUSdnRyo7GV6wXzI4cgHw2EgWJPo2dkctcGFYLhwo2ldR2AKjOZJKAdn2jxghZ1PkYKlRCEtqIRmAx4iYjhv6LZ38gJR/SpvmAtNNZPCUaeID8R+JqVrMn2G9cwCg= Received: from MW4PR21MB1907.namprd21.prod.outlook.com (2603:10b6:303:71::8) by MW2PR2101MB1082.namprd21.prod.outlook.com (2603:10b6:302:a::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4669.3; Tue, 2 Nov 2021 19:44:39 +0000 Received: from MW4PR21MB1907.namprd21.prod.outlook.com ([fe80::7d8b:5fc0:be80:ab1f]) by MW4PR21MB1907.namprd21.prod.outlook.com ([fe80::7d8b:5fc0:be80:ab1f%5]) with mapi id 15.20.4690.003; Tue, 2 Nov 2021 19:44:39 +0000 From: "Bret Barkelew" To: "devel@edk2.groups.io" , "Lindholm, Leif" , "brbarkel@microsoft.com" , Sean Brogan CC: "devel@edk2.groups.io" , Ard Biesheuvel , "Kinney, Michael D" , Liming Gao , Zhiguang Liu , Sami Mujawar , "Yao, Jiewen" , Supreeth Venkatesh , Maciej Rabeda , Jiaxin Wu , Siyuan Fu , "Ni, Ray" , Zhichao Gao , Sean Brogan Subject: Re: [EXTERNAL] Re: [edk2-devel] [PATCH v1 05/16] ArmPkg and MdePkg: Move the Arm CompilerIntrinsicsLib to MdePkg Thread-Topic: [EXTERNAL] Re: [edk2-devel] [PATCH v1 05/16] ArmPkg and MdePkg: Move the Arm CompilerIntrinsicsLib to MdePkg Thread-Index: AQHXz88kFGKTxTHCFU+qbozG3OOBpKvwpH2n Date: Tue, 2 Nov 2021 19:44:38 +0000 Message-ID: References: <20211101195648.6420-1-brbarkel@microsoft.com> <20211101195648.6420-6-brbarkel@microsoft.com> <20211102095048.op7qudfi6cgmkb55@leviathan> In-Reply-To: <20211102095048.op7qudfi6cgmkb55@leviathan> Accept-Language: en-US X-Mentions: sean.brogan@microsoft.com X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=True;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2021-11-02T19:44:31.0966173Z;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ContentBits=0;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Method=Standard authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 4219d747-4482-429e-4a2e-08d99e393513 x-ms-traffictypediagnostic: MW2PR2101MB1082: x-ld-processed: 72f988bf-86f1-41af-91ab-2d7cd011db47,ExtAddr x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:4303; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: caFUrCPtcPEWZ2VhIK7mOuOBGh13fBUtBjVMvCloCrpzejhw7nrHtiA2RCuMjLfkqpbRdRMhxTo+MAn0afpfTAb6OuXF2uJYjEjCaJc0YZfQNM+HWLKgR9APyI3u2qFx5PQfC/7RxBTUdigg83hxXFUPBRVBt2OGUijqhJ52mDJE5SnBBpno+NP2l+ARofx+5GZmF9aghDudy4pzMFnB4qGmWct12Nx4SCYO6aBOe7nRTEywoXcy8D77UPALpYYCJ5Re9RBSp7SFJUpm+CzLrmlXV695t+FqjyovakiffRRQzrSVqgEGMjOIgcMwM+RCsgPtkexQ3KXU5xRKG/8V3C9JYzXUp+NXVFcMU3LaWJBROwbeXDXjKsTxRBlZTtxNOpWDozP7FAH0B83J/EG5YSEJ3GW0++tmx/y8j5P+lyW9OI1YN5GEMb98ptadhkbRaHs+Rp9yQ7LhEItRPP2gOAKTjFjdoqb3+FdpXSyRSPD97r0gIkPudDz4eQL47GEaRV2jCgCsi6yLLZZYY6mD34neeyZXBTsqavjjLZavJQRoKmCSBB2JZgBhiZaoExZ70QwKX30KyJfm7tQ59LsNfe3f2v6zrMpOl0ncUtKlKZFBwDitEoX6E2aiaq2jcxb5E/lJ95EG0XQCog+GXhck/THCXNxABCE69g+HWcHAoQn04Rafm3Xi0DbBPYSiCMrbDUteblYQ8EdFzhMfLDID581ietyZeP5ssPlGFdVOI54AORiL9f2VboFGwcx3RkTYFAwfZbMIWpUoFLHHnp4nrhOdUS/Q8JFa4l6pmTDK44fB0FAI/3O6u7TRb6eREr5L x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MW4PR21MB1907.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(38100700002)(7416002)(7696005)(54906003)(64756008)(966005)(8990500004)(110136005)(6636002)(76116006)(316002)(6506007)(66446008)(10290500003)(8676002)(186003)(66946007)(2906002)(508600001)(66556008)(122000001)(4326008)(5660300002)(38070700005)(26005)(83380400001)(8936002)(71200400001)(107886003)(166002)(9686003)(33656002)(53546011)(52536014)(82960400001)(82950400001)(66476007)(55016002)(86362001)(491001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?Windows-1252?Q?AAK+b1QRybrxzacqynLEFmrUwk47L5MQaUsev7oCIETT191COtmb6d0h?= =?Windows-1252?Q?0ydCAXC2IKBG6yHejEt4UEKm1oSm04OAW4SC9GXSMo8w8RcsKOfolvIe?= =?Windows-1252?Q?udPR3GRCIkjJmShIZR2Zsfg2dkZd282rH8kV1+HJFyFdr7GNAUHMYZW7?= =?Windows-1252?Q?saFAwdrhS4ko0HjdwawMcDhslhmYYQjwFWWCyQ8dnvFgZR2LKqSUOZ2v?= =?Windows-1252?Q?Ah0jUFYnTAKb40ggZ76Um7Ugyue5Cxib3Ww2Qdj2DRnDhgRwMJJPpNJP?= =?Windows-1252?Q?firxQk18GJMUwkOPfuAKjTNI0/5fZhH9gWdAeMUL9USr0SaGPRGmlEtY?= =?Windows-1252?Q?hr14qYrKcG1ezA2a9LTRj6soJfB35jYvLQTcNWzPVuhkVl1Sft8uhiyQ?= =?Windows-1252?Q?/G0yvqXVHc/BC3Ac3hygipIuKTmUVoSAuhpX1Gb1pFFK4hE/yoyRdcGq?= =?Windows-1252?Q?psMb+BlIRuu0wADc0PBOU1QVvh46JdQh9Azw9dwT2SBK1geHXtqM75As?= =?Windows-1252?Q?weGDPlZwVhcv22VTdRaCYx1qFw7C71/GHuzjG+/6ZzegH1tVACCE7C3C?= =?Windows-1252?Q?mPLP35Am5nPTDg6/spNkkYyW2y9yyqNI0iIPimlHGnoJAMNd+dyZjpDg?= =?Windows-1252?Q?AJy8MmwrXuv9Eaj4d/APn3HJH3BkH4FppOzaEHkKsQSMtv3q0MwM7qpw?= =?Windows-1252?Q?SFLeWhXXA+Ca1pkK0WVB/O3A9MwOsw15QdhUZE99KZ/qNNXh8BkKlp1Z?= =?Windows-1252?Q?eAIgbNU7497qV7ar+VqmoYNKzP5HOwPUZsFI1VFMK/QD3UvLkS9MLHiZ?= =?Windows-1252?Q?FI6UelZBClHS1JPOOlk/8H5tQrCpnahsOMOVAQjZLvPdrMaiuuetEeAv?= =?Windows-1252?Q?bQh3o3nwZYauJ+eMLApcNqy0YvQ7iy7DTrjPhn87ehzedAxafWDkldHy?= =?Windows-1252?Q?IjGKa+NpuMzq+bGgRYsyEaMKoTyi/vbIMJkwG/us49hv12E1enTi3Yda?= =?Windows-1252?Q?NVi47PYaldBYYb8cIzTh6anh9oDSssKIDe98IVSTo2px6F34YXmsIjX+?= =?Windows-1252?Q?7nbAuZTLD6SWJSK24M/JOWkELLQzMGTbQKdNNfstSZJGBLBTfahSSaoR?= =?Windows-1252?Q?ZbAsq7CK9lx22eOZSpjo/bJM7xqczYPmk3QwvlbMpwCTykF9vz6KI6fD?= =?Windows-1252?Q?jqQ7tH6l3ZicMDi/oHN7uW2Yf9vHPs18OZoZjjsML53XF0xRUgd5HLA9?= =?Windows-1252?Q?uAzU7g7g0jocWnND5IIHhTr6C7nxtVBqjSnzjkAicKjToHDzzPUHdiw/?= =?Windows-1252?Q?m0ZsY5norsBxOBL1o5wekZRl5EFOJU4LDXIGBiITCwyilGXLYDkvoFQR?= =?Windows-1252?Q?3ZJa8e9BIVz1QzWQAkpQTYoE0ihsVyP4vYrALxXtFbRPiTq87S5QGoS+?= =?Windows-1252?Q?GZBBKlV2VmkvRKeQ9fPLH7VM6guM3oapw97TvPasw2GO0WsR3IXUEp9/?= =?Windows-1252?Q?910J5pzCgR1imavTo0vmk5iaccIjf9Fu2ziW7LF8R9Caz8vC1pFiZUfs?= =?Windows-1252?Q?ZuQ7P5UOpWhDXwFlGyz2770JVpffWi9lVjAfrhEY0NImVLWp4U7JFf53?= =?Windows-1252?Q?Z2EQsklQBWBoPwLIlwg5QJte9nBMnWL1iF5liO76Q8nHDN0ClpqMexLE?= =?Windows-1252?Q?V8Di5eFzDFb3yEWTblWceG+Piw3PwTBTI8Zj6cWl4lGVoI/RxtKl48z3?= =?Windows-1252?Q?+5cGoL5V/yM8tYOt6M14ZZHbhFCZ/sr3AXxxr8XZ?= MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MW4PR21MB1907.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4219d747-4482-429e-4a2e-08d99e393513 X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Nov 2021 19:44:38.9829 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: Rb7aXecANpe40PQF+zPUlLkfjmjk0GsugCXnwdd60FCzDq6Lw++O63zFv45YRQ4XAF9qNCkz841/qVkmMYCJ1Q== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW2PR2101MB1082 Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_MW4PR21MB190724CF831C862BC5915879EF8B9MW4PR21MB1907namp_" --_000_MW4PR21MB190724CF831C862BC5915879EF8B9MW4PR21MB1907namp_ Content-Type: text/plain; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable I=92d be amenable to that. @Sean? - Bret From: Leif Lindholm via groups.io Sent: Tuesday, November 2, 2021 2:51 AM To: brbarkel@microsoft.com Cc: devel@edk2.groups.io; Ard Biesheuvel; Kinney, Michael D; Liming Gao; Zhiguang Liu; Sami Mujawar; Yao, Jie= wen; Supreeth Venkatesh; Maciej Rabeda; Jiaxin W= u; Siyuan Fu; Ni, R= ay; Zhichao Gao; Sea= n Brogan Subject: [EXTERNAL] Re: [edk2-devel] [PATCH v1 05/16] ArmPkg and MdePkg: Mo= ve the Arm CompilerIntrinsicsLib to MdePkg On Mon, Nov 01, 2021 at 12:56:37 -0700, brbarkel@microsoft.com wrote: > From: Bret Barkelew > > 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=3Dhttps%3A%2F%2F= bugzilla.tianocore.org%2Fshow_bug.cgi%3Fid%3D3649&data=3D04%7C01%7CBret= .Barkelew%40microsoft.com%7C6acadbd5a03f44edf9c408d99de642ee%7C72f988bf86f1= 41af91ab2d7cd011db47%7C1%7C0%7C637714434603817528%7CUnknown%7CTWFpbGZsb3d8e= yJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&am= p;sdata=3DAmwBKl6uWTs1VYo3Hw3I3RglkhPrWXhKPkobrwQid84%3D&reserved=3D0 > Only one comment on this patch really. > ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf =3D> MdeP= kg/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 --_000_MW4PR21MB190724CF831C862BC5915879EF8B9MW4PR21MB1907namp_ Content-Type: text/html; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable

I=92d be amenable to that. @Sean?

 

- Bret

 

From: Leif Lindholm via groups.io=
Sent: Tuesday, November 2, 2021 2:51 AM
To: brbarkel@microsoft.com Cc: devel@edk2.groups.io= ; Ard Biesheuvel; Kinney, M= ichael D; Liming Gao; Zhiguang Liu; Sami Mujawar;= Yao, Jiewen; Supreeth Ven= katesh; Maciej Rabeda; Jiaxin Wu; Siyuan Fu; Ni, Ray; Zhichao Gao; Sean Brogan
Subject: [EXTERNAL] Re: [edk2-devel] [PATCH v1 05/16] ArmPkg and Mde= Pkg: Move the Arm CompilerIntrinsicsLib to MdePkg

 

On Mon, Nov 01, 2021 = at 12:56:37 -0700, brbarkel@microsoft.com wrote:
> From: Bret Barkelew <brbarkel@microsoft.com>
>
> This aligns better with Mu's philosophy around dependency structuring<= br> > and is one of the steps to enable Basecore to have zero CI dependencie= s
> on other Mu repos.
>
> REF: https://nam06.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fbugzill= a.tianocore.org%2Fshow_bug.cgi%3Fid%3D3649&amp;data=3D04%7C01%7CBret.Ba= rkelew%40microsoft.com%7C6acadbd5a03f44edf9c408d99de642ee%7C72f988bf86f141a= f91ab2d7cd011db47%7C1%7C0%7C637714434603817528%7CUnknown%7CTWFpbGZsb3d8eyJW= IjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&a= mp;sdata=3DAmwBKl6uWTs1VYo3Hw3I3RglkhPrWXhKPkobrwQid84%3D&amp;reserved= =3D0
>

Only one comment on this patch really.

>  ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf = =3D> 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




 

--_000_MW4PR21MB190724CF831C862BC5915879EF8B9MW4PR21MB1907namp_--