MattDevereau marked 9 inline comments as done. MattDevereau added inline comments.
================ Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:732 + Type *VecTyPtr = II.getType()->getPointerTo(); + IRBuilder<> Builder(II.getContext()); + Builder.SetInsertPoint(&II); ---------------- DavidTruby wrote: > Nit: I think the default template arguments should just be picked without the > `<>` I'm getting compiler errors when omitting `<>` ================ Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:750 + Value *PtrOp = II.getOperand(1); + IRBuilder<> Builder(II.getContext()); + Builder.SetInsertPoint(&II); ---------------- DavidTruby wrote: > I'm getting compiler errors when omitting <> ================ Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:765 + Type *VecTyPtr = VecOp->getType()->getPointerTo(); + IRBuilder<> Builder(II.getContext()); + Builder.SetInsertPoint(&II); ---------------- DavidTruby wrote: > I'm getting compiler errors when omitting <> Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113489/new/ https://reviews.llvm.org/D113489 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits