[clang] [llvm] [llvm][clang] Remove `llvm::OwningArrayRef` (PR #169126)

David Stone via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 22 10:04:46 PST 2025


================
@@ -22,10 +22,10 @@ struct CGDataPatchItem {
   // Where to patch.
   uint64_t Pos;
   // Source data.
-  OwningArrayRef<uint64_t> D;
+  std::vector<uint64_t> D;
----------------
davidstone wrote:

I haven't audited all of the uses of `CGDataPatchItem` to ensure that none of the clients rely on a reference to the data being stable on a move. That is guaranteed by `OwningArrayRef` and `std::vector` but not `SmallVector`.

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


More information about the llvm-commits mailing list