From d5dab23622a38415971b4fe24e0dffae53365bf0 Mon Sep 17 00:00:00 2001 From: Ramkumar Ramachandra Date: Thu, 22 Aug 2024 13:41:11 +0100 Subject: [PATCH] LV/Builder: inline VPBuilder::createICmp (NFC) Inline VPBuilder::createICmp in the header, in line with the other VPBuilder functions. --- llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h | 7 ++++++- llvm/lib/Transforms/Vectorize/LoopVectorize.cpp | 8 -------- 2 files changed, 6 insertions(+), 9 deletions(-) diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h b/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h index 3bb7a8e651a3f..968cab7bd77b1 100644 --- a/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h +++ b/llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h @@ -200,7 +200,12 @@ class VPBuilder { /// and \p B. /// TODO: add createFCmp when needed. VPValue *createICmp(CmpInst::Predicate Pred, VPValue *A, VPValue *B, - DebugLoc DL = {}, const Twine &Name = ""); + DebugLoc DL = {}, const Twine &Name = "") { + assert(Pred >= CmpInst::FIRST_ICMP_PREDICATE && + Pred <= CmpInst::LAST_ICMP_PREDICATE && "invalid predicate"); + return tryInsertInstruction( + new VPInstruction(Instruction::ICmp, Pred, A, B, DL, Name)); + } //===--------------------------------------------------------------------===// // RAII helpers. diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp index 2145bb8c9ca87..6c9810b1df87f 100644 --- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp +++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp @@ -6903,14 +6903,6 @@ void LoopVectorizationCostModel::collectInLoopReductions() { } } -VPValue *VPBuilder::createICmp(CmpInst::Predicate Pred, VPValue *A, VPValue *B, - DebugLoc DL, const Twine &Name) { - assert(Pred >= CmpInst::FIRST_ICMP_PREDICATE && - Pred <= CmpInst::LAST_ICMP_PREDICATE && "invalid predicate"); - return tryInsertInstruction( - new VPInstruction(Instruction::ICmp, Pred, A, B, DL, Name)); -} - // This function will select a scalable VF if the target supports scalable // vectors and a fixed one otherwise. // TODO: we could return a pair of values that specify the max VF and