This is an archive of the discontinued LLVM Phabricator instance.

[Headers][doc] Add SHA1/SHA256 intrinsic descriptions
ClosedPublic

Authored by probinson on Jul 20 2023, 9:52 AM.

Details

Summary

I didn't include pseudo-code, because it would be long and complicated, probably not tell the whole story, and these are _extremely_ specialized instructions. I can have a go at it if reviewers insist.

FTR this should be the last of the intrinsic description reviews I plan to post. I still need to double-check my list, but I'm pretty sure I got them all.

Diff Detail

Event Timeline

probinson created this revision.Jul 20 2023, 9:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 20 2023, 9:52 AM
probinson requested review of this revision.Jul 20 2023, 9:52 AM
pengfei accepted this revision.Jul 20 2023, 7:03 PM

LGTM.

This revision is now accepted and ready to land.Jul 20 2023, 7:03 PM

Thanks!

The pre-merge check caught mistakes in the \param commands, which I fixed before pushing.

This revision was landed with ongoing or failed builds.Jul 21 2023, 2:27 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJul 21 2023, 2:27 PM