This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt][AArch64] Correct how FMV use ifunc resolver abi.
ClosedPublic

Authored by ilinpv on Jul 11 2023, 3:46 PM.

Details

Summary

The patch fixes second argument of FMV resolvers, it is pointer to an
extendible struct containing hwcap and hwcap2 not a unsigned long hwcap2.
Also fixes FMV features caching in resolver.

Diff Detail

Event Timeline

ilinpv created this revision.Jul 11 2023, 3:46 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 11 2023, 3:46 PM
ilinpv requested review of this revision.Jul 11 2023, 3:46 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 11 2023, 3:46 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
enh accepted this revision.Jul 11 2023, 3:57 PM
This revision is now accepted and ready to land.Jul 11 2023, 3:57 PM
pirama added a subscriber: pirama.Jul 11 2023, 3:59 PM
MaskRay accepted this revision.Jul 11 2023, 4:05 PM
MaskRay added a subscriber: MaskRay.

Looks good. This matches https://github.com/ARM-software/abi-aa/blob/main/sysvabi64/sysvabi64.rst#gnu-c-library-ifunc-interface

The patch fixes second argument of FMV resolvers, it is pointer to an

Nit: avoid using abbreviation for less common terms like FMV. Just expand it.