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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
