[PATCH] D140896: [WIP] Move from llvm::makeArrayRef to ArrayRef deduction guides

Jacques Pienaar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 3 11:15:59 PST 2023


jpienaar added inline comments.


================
Comment at: llvm/include/llvm/ADT/ArrayRef.h:502
+  /// Deduction guide to construct an ArrayRef from a C array.
+  template <typename T, size_t N> ArrayRef(const T (&Arr)[N]) -> ArrayRef<T>;
 
----------------
mehdi_amini wrote:
> Can we keep the makeArrayRef functions for now and mark them deprecated?
+1 that would also allow for this change to broken up so that the deduction guides land first and then the updates (potentially even 3 hops, 1) add guides, 2) update, 3) mark deprecated - 1 & 2 could be combined but it'll greatly simplify review/enable sharding it)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140896/new/

https://reviews.llvm.org/D140896



More information about the cfe-commits mailing list