This is an archive of the discontinued LLVM Phabricator instance.

[Headers] Add some intrinsic function descriptions to immintrin.h
ClosedPublic

Authored by probinson on Apr 3 2023, 11:25 AM.

Diff Detail

Event Timeline

probinson created this revision.Apr 3 2023, 11:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 3 2023, 11:25 AM
probinson requested review of this revision.Apr 3 2023, 11:25 AM

FTR, I'll be working my way through a bunch of intrinsics over the next month or so, trying not to do too many at once. Mostly AVX2 but also some others.

pengfei accepted this revision.Apr 4 2023, 12:36 AM

LGTM, thanks for working on it!

clang/lib/Headers/immintrin.h
294

memory location

294

Nit: I saw others prefer to A pointer

309

ditto.

324

ditto.

414

Nit: I saw others prefer to A 32-bit integer value.

This revision is now accepted and ready to land.Apr 4 2023, 12:36 AM
This revision was landed with ongoing or failed builds.Apr 4 2023, 6:26 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 4 2023, 6:26 AM