[libcxx-commits] [PATCH] D80057: [libc++][NFCI] Optimization to std::~unique_ptr

[JF Bastien via Phabricator via libcxx-commits]

jfb added a comment.

Fine by me as long as it's heavily publicized, so folks realize what's being changed. I'd like to see a measured perf upside proving that this is worth doing, and I'd have hoped that it would have been easy enough to provide. Does the compiler usually remove the store anyways, in which case this patch isn't actually doing anything useful?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80057