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

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa9e08bc7c1e5: [AArch64][SVE] InstCombine llvm.aarch64.sve.sel to select (authored by MattDevereau). Changed prior to commit: https://reviews.llvm.

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

peterwaller-arm added a comment. In D121792#3388585 , @dmgreen wrote: > Why do we have llvm.aarch64.sve.sel if we are always going to replace it with > a select? Why not remove llvm.aarch64.sve.sel entirely? I'm not the key decisionmaker here, but hopef

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

dmgreen added a comment. Why do we have llvm.aarch64.sve.sel if we are always going to replace it with a select? Why not remove llvm.aarch64.sve.sel entirely? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121792/new/ https://reviews.llvm.org/D1217

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

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<> Bui

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

MattDevereau created this revision. MattDevereau added reviewers: peterwaller-arm, paulwalker-arm, DavidTruby, bsmith. Herald added subscribers: ctetreau, psnobl, hiraditya, kristof.beyls, tschuett. Herald added a reviewer: efriedma. Herald added a project: All. MattDevereau requested review of th