[PATCH] D55517: Remove `_VSTD`
Eric Fiselier via Phabricator
reviews at reviews.llvm.org
Mon Dec 10 11:59:23 PST 2018
EricWF added a comment.
In D55517#1325917 <https://reviews.llvm.org/D55517#1325917>, @jfb wrote:
> I think your commit message is fun and terse, but it doesn't say why you're actually correct. You're explaining it here, Marshall has voiced concerns about downsides. I think your commit message should explain this and say why you think the downsides aren't relevant. That makes it easier to go back to your change in the future and understand why the change was OK without looking at this discussion.
How's the new commit message?
Repository:
rCXX libc++
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55517/new/
https://reviews.llvm.org/D55517
More information about the libcxx-commits
mailing list