[PATCH] D55517: Remove `_VSTD`
JF Bastien via Phabricator
reviews at reviews.llvm.org
Mon Dec 10 12:16:52 PST 2018
jfb added a comment.
In D55517#1325994 <https://reviews.llvm.org/D55517#1325994>, @EricWF wrote:
> 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?
Looks great, ty!
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