[libcxx-commits] [libcxx] [libcxx] Remove Redundant Reset in ~basic_string (PR #164718)

[Louis Dionne via libcxx-commits]

https://github.com/ldionne commented:

Thanks for the report. I had also noticed some change in benchmarks I've been running and I thought it might be related to this patch, but I was then unable to reproduce the regression so Nikolas and I had kind of assumed it wasn't an actual regression. Perhaps it was. This is where e.g. LNT would come in really handy -- let's hope we can unblock that soon!

I think it would be great to have a regression test or a benchmark for this, but I understand it might be difficult to capture.

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