[llvm] [RemoveDIs] Add iterator-taking constructors and Create methods (PR #82778)

Orlando Cazalet-Hyams via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 26 02:03:06 PST 2024


================
@@ -1730,6 +1896,16 @@ class SelectInst : public Instruction {
   SelectInst *cloneImpl() const;
 
 public:
+  static SelectInst *Create(Value *C, Value *S1, Value *S2,
+                            const Twine &NameStr,
+                            BasicBlock::iterator InsertBefore,
+                            Instruction *MDFrom = nullptr) {
+    SelectInst *Sel = new (3) SelectInst(C, S1, S2, NameStr, InsertBefore);
----------------
OCHyams wrote:

nit: existing format is `new(3)` (yours is `new (3)` with a space). Feel free to ignore if clang-format is happy.

https://github.com/llvm/llvm-project/pull/82778


More information about the llvm-commits mailing list