Page MenuHomePhabricator

[clang] use getCommonSugar in an assortment of places
Needs ReviewPublic

Authored by mizvekov on Oct 10 2021, 6:36 AM.

Details

Reviewers
rsmith
Group Reviewers
Restricted Project
Summary

For this patch, a simple search was performed for patterns where there are
two types (usually an LHS and an RHS) which are structurally the same, and there
is some result type which is resolved as either one of them (typically LHS for
consistency).

We change those cases to resolve as the common sugared type between those two,
utilizing the new infrastructure created for this purpose.

Depends on D111283

Signed-off-by: Matheus Izvekov <mizvekov@gmail.com>

Diff Detail

Event Timeline

mizvekov created this revision.Oct 10 2021, 6:36 AM
mizvekov retitled this revision from [clang] WIP: Use getCommonSugar in an assortment of places to [clang] WIP: use getCommonSugar in an assortment of places.Oct 10 2021, 6:37 AM
mizvekov published this revision for review.Oct 12 2021, 4:00 PM
mizvekov retitled this revision from [clang] WIP: use getCommonSugar in an assortment of places to [clang] use getCommonSugar in an assortment of places.
Herald added projects: Restricted Project, Restricted Project, Restricted Project. · View Herald TranscriptOct 12 2021, 4:00 PM
Herald added subscribers: cfe-commits, Restricted Project. · View Herald Transcript
mizvekov edited the summary of this revision. (Show Details)Sun, Oct 31, 9:03 AM
rsmith accepted this revision.Fri, Nov 12, 2:03 PM
This revision is now accepted and ready to land.Fri, Nov 12, 2:03 PM
mizvekov updated this revision to Diff 387579.Tue, Nov 16, 5:01 AM
  • Run libcxx CI
Herald added a project: Restricted Project. · View Herald TranscriptTue, Nov 16, 5:01 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
This revision now requires review to proceed.Tue, Nov 16, 5:01 AM