[PATCH] D83248: [NFCI][IR] Introduce CallBase::Create() wrapper

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri marked 2 inline comments as done.
lebedev.ri added inline comments.


================
Comment at: llvm/include/llvm/IR/InstrTypes.h:1145
+  static CallBase *Create(CallBase *CB, ArrayRef<OperandBundleDef> Bundles,
+                          Instruction *InsertPt = nullptr);
+
----------------
nickdesaulniers wrote:
> No call sites make use of this default value (`nullptr`). Is it the right value?  I'm not sure if you can default it to `CB`, but if not, should it just be a required parameter with no default value?
This is a direct wrapper over all the subclasses `Create()` function with the same signature.
They all follow this pattern. I'm not sure why we should deviate here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83248