Skip to content

Commit

Permalink
[IR][NFC] Cleanup CmpInst signatures / code docs (#86441)
Browse files Browse the repository at this point in the history
Change param names to recommended upper case format for static methods
in CmpInst for consistency
Implement suggestion from @dtcxzyw.

cc @dtcxzyw @tschuett
  • Loading branch information
marcauberer committed Mar 28, 2024
1 parent c9db031 commit a495cfb
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions llvm/include/llvm/IR/InstrTypes.h
Expand Up @@ -1032,31 +1032,31 @@ class CmpInst : public Instruction {
/// the two operands. Insert the instruction into a BasicBlock right before
/// the specified instruction.
/// Create a CmpInst
static CmpInst *Create(OtherOps Op, Predicate predicate, Value *S1, Value *S2,
static CmpInst *Create(OtherOps Op, Predicate Pred, Value *S1, Value *S2,
const Twine &Name, BasicBlock::iterator InsertBefore);

/// Construct a compare instruction, given the opcode, the predicate and
/// the two operands. Optionally (if InstBefore is specified) insert the
/// instruction into a BasicBlock right before the specified instruction.
/// The specified Instruction is allowed to be a dereferenced end iterator.
/// Create a CmpInst
static CmpInst *Create(OtherOps Op,
Predicate predicate, Value *S1,
Value *S2, const Twine &Name = "",
static CmpInst *Create(OtherOps Op, Predicate Pred, Value *S1, Value *S2,
const Twine &Name = "",
Instruction *InsertBefore = nullptr);

/// Construct a compare instruction, given the opcode, the predicate and the
/// two operands. Also automatically insert this instruction to the end of
/// the BasicBlock specified.
/// Create a CmpInst
static CmpInst *Create(OtherOps Op, Predicate predicate, Value *S1,
Value *S2, const Twine &Name, BasicBlock *InsertAtEnd);
static CmpInst *Create(OtherOps Op, Predicate Pred, Value *S1, Value *S2,
const Twine &Name, BasicBlock *InsertAtEnd);

/// Construct a compare instruction, given the opcode, the predicate,
/// the two operands and the instruction to copy the flags from. Optionally
/// (if InstBefore is specified) insert the instruction into a BasicBlock
/// right before the specified instruction. The specified Instruction is
/// allowed to be a dereferenced end iterator. Create a CmpInst
/// allowed to be a dereferenced end iterator.
/// Create a CmpInst
static CmpInst *CreateWithCopiedFlags(OtherOps Op, Predicate Pred, Value *S1,
Value *S2,
const Instruction *FlagsSource,
Expand Down

0 comments on commit a495cfb

Please sign in to comment.