[PATCH] D121792: [AArch64][SVE] InstCombine llvm.aarch64.sve.sel to select

[Sander de Smalen via Phabricator via cfe-commits]

sdesmalen accepted this revision.
sdesmalen added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:530-531
+                                                 IntrinsicInst &II) {
+  IRBuilder<> Builder(II.getContext());
+  Builder.SetInsertPoint(&II);
+  auto Select = Builder.CreateSelect(II.getOperand(0), II.getOperand(1),
----------------
nit: Does [[ https://llvm.org/doxygen/classllvm_1_1IRBuilder.html#a5736f43e3fc238e241314f480ff4d809 | this ]] constructor avoid doing this in two steps?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121792