Page MenuHomePhabricator

[DAGCombine] Replace std::monostate equivalent in DAGCombiner.cpp
ClosedPublic

Authored by jloser on Aug 12 2022, 7:28 AM.

Details

Summary

Remove the UnitT type and operators in favor of using std::monostate
directly.

Diff Detail

Event Timeline

jloser created this revision.Aug 12 2022, 7:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 12 2022, 7:28 AM
jloser requested review of this revision.Aug 12 2022, 7:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 12 2022, 7:28 AM
This revision is now accepted and ready to land.Aug 12 2022, 9:47 AM
jloser updated this revision to Diff 452264.Aug 12 2022, 11:53 AM

Run git-clang-format HEAD~1. If CI passes, I'll land this.

jloser updated this revision to Diff 452301.Aug 12 2022, 2:10 PM

Fix reference to Unit. Use std::monostate{} instead.

jloser updated this revision to Diff 452323.Aug 12 2022, 4:12 PM

Fix another use of Unit in favor of std::monostate{}.