[libcxx-commits] [PATCH] D62428: [libcxx] Slightly improved policy for handling experimental features

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

Folks, I just spoke to Marshall over the phone and I believe one of the reasons for the opposition is that I didn't make it clear we were not warning on ALL uses of experimental features. The new proposal only adds deprecation warnings when we standardize a feature and we know FOR SURE we're going to remove the experimental version in the near future. I didn't make that change from the initial proposal clear, and I apologize for that. With that thing being clear, the proposal seemed to be more reasonable.

Now, it's not to say that this clears out ALL opposition. We agreed to hold a wider-audience discussion about the place of warnings (and deprecation warnings in particular) inside the compiler versus a separate tool (e.g. `clang-tidy`). I'm very much looking forward to that discussion because right now we only have a hammer (the compiler), and as a result everything looks like a nail. However, we decided not to make that discussion a prerequisite for the policy that is the object of this patch.

Marshall, I'm looking forward to your comments on the proposal, and to the discussion on `cfe-dev` about deprecation warnings in the compiler in general!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62428