[PATCH] D31937: Move value type list from TargetRegisterClass to TargetRegisterInfo

[Krzysztof Parzyszek via Phabricator via llvm-commits]

kparzysz added inline comments.


================
Comment at: include/llvm/Target/TargetRegisterInfo.h:320
+  /// Return true if the given TargetRegisterClass has the ValueType T.
+  bool hasType(const TargetRegisterClass &RC, MVT T) const {
+    for (int i = 0; RC.VTs[i] != MVT::Other; ++i)
----------------
MatzeB wrote:
> The name TargetRegisterInfo::hasType feels odd as this isn't a general property of TargetRegisterInfo. Maybe rename to `regclassHasType()`?
> 
> Similar with the next two functions (`regclassTypes_begin()` maybe?).
How about `isTypeLegalForClass` and `regclassvt_begin`, `regclassvt_end`?


Repository:
  rL LLVM

https://reviews.llvm.org/D31937