This is an archive of the discontinued LLVM Phabricator instance.

[ADT] Simplify hashing for tuples
ClosedPublic

Authored by jloser on Oct 27 2022, 8:10 AM.

Details

Summary

Instead of using std::index_sequence with a helper function template to access
each element in the tuple, leverage std::apply from C++17 to do the heavy
lifting for us.

Diff Detail

Event Timeline

jloser created this revision.Oct 27 2022, 8:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 27 2022, 8:10 AM
Herald added a subscriber: StephenFan. · View Herald Transcript
jloser requested review of this revision.Oct 27 2022, 8:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 27 2022, 8:10 AM
dblaikie accepted this revision.Oct 27 2022, 12:42 PM

Lovely :)

This revision is now accepted and ready to land.Oct 27 2022, 12:42 PM
This revision was automatically updated to reflect the committed changes.
kazu added a comment.Oct 27 2022, 2:22 PM

Thank you for the cleanup!