[PATCH] D47770: [MVT][SVE] Add EVT strings and Type mapping

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 7 13:58:03 PST 2019


greened added inline comments.


================
Comment at: lib/CodeGen/ValueTypes.cpp:193
   case MVT::v8f64:   return "v8f64";
+  case MVT::nxv1i1:  return "nxv1i1";
+  case MVT::nxv2i1:  return "nxv2i1";
----------------
I can't find where these are defined, but given that the IR type is <scalable v4f32> should these names reflect that?  "nx" is nice and short but may be confusing in that the mapping from "scalable" to "nx" isn't necessarily obvious.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D47770/new/

https://reviews.llvm.org/D47770





More information about the llvm-commits mailing list