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

[Philip Reames via Phabricator via llvm-commits]

reames added inline comments.


================
Comment at: llvm/include/llvm/IR/IRBuilder.h:898
+  /// at runtime.
+  Value *CreateRuntimeElementCount(Type *DstType, ElementCount EC);
+
----------------
luke wrote:
> paulwalker-arm wrote:
> > 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 :)
> +1, `CreateElementCount` makes sense to me. 
I'm going to take this naming suggestion and run with it.  Seems reasonable to me as well.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145945