[PATCH] D98874: [TTI] Return a TypeSize from getRegisterBitWidth.
Paul Walker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 19 09:57:13 PDT 2021
paulwalker-arm accepted this revision.
paulwalker-arm added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/lib/Target/ARM/ARMTargetTransformInfo.h:157-158
+ case TargetTransformInfo::RGK_FixedWidthVector:
+ return TypeSize::getFixed((ST->hasNEON() || ST->hasMVEIntegerOps()) ? 128
+ : 0);
+ case TargetTransformInfo::RGK_ScalableVector:
----------------
Happy as is but do think the following looks nicer:
```
if (ST->hasNEON())
return TypeSize::getFixed(128);
if (ST->hasMVEIntegerOps())
return TypeSize::getFixed(128);
return TypeSize::getFixed(0);
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98874/new/
https://reviews.llvm.org/D98874
More information about the llvm-commits
mailing list