[PATCH] D47073: Document and Enforce new Host Compiler Policy

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 16 10:31:24 PST 2019


jfb added a comment.

In D47073#1360116 <https://reviews.llvm.org/D47073#1360116>, @mehdi_amini wrote:

> In D47073#1360077 <https://reviews.llvm.org/D47073#1360077>, @jfb wrote:
>
> > That's incorrect. I don't care what the policy is as long as it's sane. 3 years was proposed on the RFC, we went with it. Then nuance was asked for on this review, we went with it. Then it was asked that time was made a soft constraint, we went with it. Then more bikeshed happened.
>
>
> This is a misrepresentation of what happened. 
>  I can give you an alternative version but your attitude in this thread does not lead me to think you'd listen.
>
> > The version was absolutely chosen based on agreement that we're moving off C++11, and C++14 is the next version over.
>
> This is not true.
>
> If all you want is C++14 and a warning in LLVM 8.0, the path was fairly clear to me: properly address reviews, don't ignore questions when they don't serve your agenda, and finally don't rush patches when there are controversial points unadressed.
>  If you think that's how an open source project should work, I think we'll run into more dead-ends.


This is not a constructive response, and I don't intend to keep replying to this thread. As I've said, this review is past its usefulness. In the follow-up, please avoid ascribing ill-intent to what I'm doing. Specifically, "misrepresentation", "you'd listen", "ignore questions", "serve your agenda", "rush patches", "controversial", are all phrases that I find don't lead to a productive discussion.


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

https://reviews.llvm.org/D47073





More information about the llvm-commits mailing list