[llvm-commits] [llvm] r164093 - /llvm/trunk/include/llvm/Instructions.h

[Craig Topper]

Added in r164101.

On Mon, Sep 17, 2012 at 9:07 PM, Sean Silva <silvas at purdue.edu> wrote:

> When is this going to be added to the coding standards?
>
> --Sean Silva
>
> On Mon, Sep 17, 2012 at 11:25 PM, Craig Topper <craig.topper at gmail.com>
> wrote:
> > Author: ctopper
> > Date: Mon Sep 17 22:25:49 2012
> > New Revision: 164093
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=164093&view=rev
> > Log:
> > Mark unimplemented operator new as LLVM_DELETED_FUNCTION.
> >
> > Modified:
> >     llvm/trunk/include/llvm/Instructions.h
> >
> > Modified: llvm/trunk/include/llvm/Instructions.h
> > URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Instructions.h?rev=164093&r1=164092&r2=164093&view=diff
> >
> ==============================================================================
> > --- llvm/trunk/include/llvm/Instructions.h (original)
> > +++ llvm/trunk/include/llvm/Instructions.h Mon Sep 17 22:25:49 2012
> > @@ -255,7 +255,7 @@
> >  /// StoreInst - an instruction for storing to memory
> >  ///
> >  class StoreInst : public Instruction {
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >    void AssertOK();
> >  protected:
> >    virtual StoreInst *clone_impl() const;
> > @@ -382,7 +382,7 @@
> >  /// FenceInst - an instruction for ordering other memory operations
> >  ///
> >  class FenceInst : public Instruction {
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >    void Init(AtomicOrdering Ordering, SynchronizationScope SynchScope);
> >  protected:
> >    virtual FenceInst *clone_impl() const;
> > @@ -450,7 +450,7 @@
> >  /// there.  Returns the value that was loaded.
> >  ///
> >  class AtomicCmpXchgInst : public Instruction {
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >    void Init(Value *Ptr, Value *Cmp, Value *NewVal,
> >              AtomicOrdering Ordering, SynchronizationScope SynchScope);
> >  protected:
> > @@ -557,7 +557,7 @@
> >  /// the old value.
> >  ///
> >  class AtomicRMWInst : public Instruction {
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >  protected:
> >    virtual AtomicRMWInst *clone_impl() const;
> >  public:
> > @@ -1839,7 +1839,7 @@
> >  class InsertValueInst : public Instruction {
> >    SmallVector<unsigned, 4> Indices;
> >
> > -  void *operator new(size_t, unsigned); // Do not implement
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >    InsertValueInst(const InsertValueInst &IVI);
> >    void init(Value *Agg, Value *Val, ArrayRef<unsigned> Idxs,
> >              const Twine &NameStr);
> > @@ -1970,7 +1970,7 @@
> >  // scientist's overactive imagination.
> >  //
> >  class PHINode : public Instruction {
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >    /// ReservedSpace - The number of operands actually allocated.
>  NumOperands is
> >    /// the number actually in use.
> >    unsigned ReservedSpace;
> > @@ -2178,7 +2178,7 @@
> >  public:
> >    enum ClauseType { Catch, Filter };
> >  private:
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >    // Allocate space for exactly zero operands.
> >    void *operator new(size_t s) {
> >      return User::operator new(s, 0);
> > @@ -2445,7 +2445,7 @@
> >  /// SwitchInst - Multiway switch
> >  ///
> >  class SwitchInst : public TerminatorInst {
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >    unsigned ReservedSpace;
> >    // Operands format:
> >    // Operand[0]    = Value to switch on
> > @@ -2857,7 +2857,7 @@
> >  /// IndirectBrInst - Indirect Branch Instruction.
> >  ///
> >  class IndirectBrInst : public TerminatorInst {
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >    unsigned ReservedSpace;
> >    // Operand[0]    = Value to switch on
> >    // Operand[1]    = Default basic block destination
> > @@ -3251,7 +3251,7 @@
> >  /// end of the block cannot be reached.
> >  ///
> >  class UnreachableInst : public TerminatorInst {
> > -  void *operator new(size_t, unsigned);  // DO NOT IMPLEMENT
> > +  void *operator new(size_t, unsigned) LLVM_DELETED_FUNCTION;
> >  protected:
> >    virtual UnreachableInst *clone_impl() const;
> >
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



-- 
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120917/37e01474/attachment.html>