[llvm] [NFC][RemoveDIs] Have CreateNeg only accept iterators, template for other patches (PR #82999)

Jeremy Morse via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 26 05:18:10 PST 2024


================
@@ -270,7 +270,7 @@ static BinaryOperator *CreateMul(Value *S1, Value *S2, const Twine &Name,
 }
 
 static Instruction *CreateNeg(Value *S1, const Twine &Name,
-                              Instruction *InsertBefore, Value *FlagsOp) {
+                              BasicBlock::iterator InsertBefore, Value *FlagsOp) {
   if (S1->getType()->isIntOrIntVectorTy())
     return BinaryOperator::CreateNeg(S1, Name, InsertBefore);
----------------
jmorse wrote:

It looks like I missed adding the definition of the iterator-taking CreateNeg in 76dd4bc036, hence this presents in the diff as the argument-type changing instead of the deletion of the `Instruction *` taking function. That's my bad; but shouldn't be the same for the other scenarios where the `Instruction *` taking functions get removed.

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


More information about the llvm-commits mailing list