On Tue, Mar 15, 2022 at 03:27:58PM +0000, Michael D Kinney wrote: > Hi Gerd, > > Thanks for working on this. I raised this topic in the new TianoCore build > and CI > meeting yesterday because the intrinsic libs are closely tied to the > toolchains > supported by TianoCore. > > I agree with the concept of moving the compiler intrinsics to a library in > the MdePkg. > > + Sean Brogan > > Sean mentioned that he has some addition work on intrinsic libs for Visual > Studio > toolchains. It would be good to see if this set of changes is aligned with > those > efforts.
Having a single place where all compiler intrinsics are located should simplify this kind of changes. > I see a few items that might be good to clean up at some point > * fltused should go into its own file. Not related to the strcmp() API > * I see size_t defined in multiple places using different mappings. Would be > good to > define that in a single location. Yes, sure. Both are valid points. I see this series as first step, consolidating the existing code in a singe place with (almost) no actual code changes. There surely will be incremental improvements and cleanups later on. I will need a few more intrinsic functions when moving edk2 to openssl 3.0 for example. take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#87628): https://edk2.groups.io/g/devel/message/87628 Mute This Topic: https://groups.io/mt/89471210/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
