[PATCH] D122032: [VP] Make VectorBuilder take IRBuilderBase instead of IRBuilder<>

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 13:23:57 PDT 2022


craig.topper created this revision.
craig.topper added reviewers: simoll, frasercrmck, rogfer01.
Herald added a subscriber: dexonsmith.
Herald added a project: All.
craig.topper requested review of this revision.
Herald added a project: LLVM.

IRBuilder<> assumes specific values for template parameters.
IRBuilderBase abstracts away the template.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D122032

Files:
  llvm/include/llvm/IR/VectorBuilder.h


Index: llvm/include/llvm/IR/VectorBuilder.h
===================================================================
--- llvm/include/llvm/IR/VectorBuilder.h
+++ llvm/include/llvm/IR/VectorBuilder.h
@@ -36,7 +36,7 @@
   };
 
 private:
-  IRBuilder<> &Builder;
+  IRBuilderBase &Builder;
   Behavior ErrorHandling;
 
   // Explicit mask parameter.
@@ -58,7 +58,7 @@
   }
 
 public:
-  VectorBuilder(IRBuilder<> &Builder,
+  VectorBuilder(IRBuilderBase &Builder,
                 Behavior ErrorHandling = Behavior::ReportAndAbort)
       : Builder(Builder), ErrorHandling(ErrorHandling), Mask(nullptr),
         ExplicitVectorLength(nullptr),


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D122032.416591.patch
Type: text/x-patch
Size: 636 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220318/57780308/attachment.bin>


More information about the llvm-commits mailing list