[PATCH] D145945: [IRBuilder] Add utilities for materializing scalable values

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added inline comments.


================
Comment at: llvm/include/llvm/IR/IRBuilder.h:898
+  /// at runtime.
+  Value *CreateRuntimeElementCount(Type *DstType, ElementCount EC);
+
----------------
I guess this is bike shedding but given IRBuilder has functions like `getInt16()` to return the IR form on a 16-bit integer value, how about just `CreateElementCount` that returns the IR form of ElementCount?

Not isolated to this patch but in general I dislike the use of `runtime` when naming things we've already named in order to allow for fixed length and scalable vectors.  By which I mean I don't really consider CompileTimeElementCount/TypeSize as something that exists. But then perhaps I'm just being too picky :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145945