[PATCH] D62202: Work around a Visual C++ bug

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 21 12:57:36 PDT 2019


rnk added a comment.

In D62202#1510522 <https://reviews.llvm.org/D62202#1510522>, @dblaikie wrote:

> Yeah, if we're going this way I'd certainly advocate having a comment of some kind explaining why it's this way so it doesn't regress.


In the past, I've found these sorts of comments to be pretty low value. They don't do anything to help the reader understand what the code is trying to do. If we want this thing to not regress, someone who cares can set up a buildbot for it. I think it's more likely that by the time somebody does this again, Paul and other users of Visual C++ will have upgraded to 14.8. Or, they'll send another one of these changes.


Repository:
  rC Clang

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

https://reviews.llvm.org/D62202





More information about the cfe-commits mailing list