This is an archive of the discontinued LLVM Phabricator instance.

Const correctness for TTI::getRegisterBitWidth
ClosedPublic

Authored by dneilson on Jun 5 2017, 11:26 AM.

Details

Summary

The method TargetTransformInfo::getRegisterBitWidth() is declared const, but the type erasing implementation classes (TargetTransformInfo::Concept & TargetTransformInfo::Model) that were introduced by Chandler in https://reviews.llvm.org/D7293 do not have the method declared const. This is an NFC to tidy up the const consistency between TTI and its implementation.

Diff Detail

Repository
rL LLVM

Event Timeline

dneilson created this revision.Jun 5 2017, 11:26 AM

Please note that I do not have commit access, yet. So, if this looks good then please upstream.

reames accepted this revision.Jun 8 2017, 10:56 AM
reames added a subscriber: reames.

Assuming this compiles cleanly, LGTM

This revision is now accepted and ready to land.Jun 8 2017, 10:56 AM
This revision was automatically updated to reflect the committed changes.