This is an archive of the discontinued LLVM Phabricator instance.

[SVE] Remove TypeSize comparison operators
ClosedPublic

Authored by david-arm on Oct 26 2020, 8:24 AM.

Details

Summary

All known instances in the code where we relied upon the TypeSize
comparison operators have now been changed to either use scalar
integer comparisons or one of the TypeSize::isKnownXY functions.
It is now safe to remove the comparison operators.

Diff Detail

Event Timeline

david-arm created this revision.Oct 26 2020, 8:24 AM
Herald added a project: Restricted Project. · View Herald Transcript
david-arm requested review of this revision.Oct 26 2020, 8:24 AM

Is this the right way to go here? I've assumed we want to remove the operators, but perhaps they should be marked as deprecated and left in for a while first?

efriedma accepted this revision.Oct 27 2020, 1:43 PM

LGTM

I don't think this is likely to significantly impact most out-of-tree users; no need to temporarily deprecate.

This revision is now accepted and ready to land.Oct 27 2020, 1:43 PM
This revision was landed with ongoing or failed builds.Oct 29 2020, 7:32 AM
This revision was automatically updated to reflect the committed changes.