[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