ADT: Use delete[] to delete the array owned by OwningArrayRef, as we created it with new[].

pcc · pcc · commit e32cf71ecc42 · 2016-12-13T20:30:12.000Z
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@289582 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/include/llvm/ADT/ArrayRef.h b/include/llvm/ADT/ArrayRef.h
@@ -424,12 +424,12 @@ namespace llvm {
     }
     OwningArrayRef(OwningArrayRef &&Other) { *this = Other; }
     OwningArrayRef &operator=(OwningArrayRef &&Other) {
-      delete this->data();
+      delete[] this->data();
       this->MutableArrayRef<T>::operator=(Other);
       Other.MutableArrayRef<T>::operator=(MutableArrayRef<T>());
       return *this;
     }
-    ~OwningArrayRef() { delete this->data(); }
+    ~OwningArrayRef() { delete[] this->data(); }
   };
 
   /// @name ArrayRef Convenience constructors

Original file line number	Diff line number	Diff line change
`@@ -424,12 +424,12 @@ namespace llvm {`
`424`	`424`	`}`
`425`	`425`	`OwningArrayRef(OwningArrayRef &&Other) { *this = Other; }`
`426`	`426`	`OwningArrayRef &operator=(OwningArrayRef &&Other) {`
`427`		`- delete this->data();`
	`427`	`+ delete[] this->data();`
`428`	`428`	`this->MutableArrayRef<T>::operator=(Other);`
`429`	`429`	`Other.MutableArrayRef<T>::operator=(MutableArrayRef<T>());`
`430`	`430`	`return *this;`
`431`	`431`	`}`
`432`		`- ~OwningArrayRef() { delete this->data(); }`
	`432`	`+ ~OwningArrayRef() { delete[] this->data(); }`
`433`	`433`	`};`
`434`	`434`
`435`	`435`	`/// @name ArrayRef Convenience constructors`