[libcxx-commits] [PATCH] D61109: Remove incorrect explicit instantiation declarations for valarray

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added inline comments.


================
Comment at: src/valarray.cpp:13
 
+#if _LIBCPP_ABI_VERSION == 1
 template valarray<size_t>::valarray(size_t);
----------------
rsmith wrote:
> ldionne wrote:
> > Can you add a comment explaining why we're doing this here?
> Done, but I'm not sure why these are removed from the V2 ABI so I'm afraid it's a little vague.
Well, they're removed because they will never be used (since there's no explicit instantiation declaration), right? My understanding is that the only reason for providing them in ABI v1 is to avoid removing symbols (and code) that already-compiled binaries might depend on, however newly compiled code will never use it. If that is incorrect, then I did not understand this patch.


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

https://reviews.llvm.org/D61109