Page MenuHomePhabricator

[clang] deprecate frelaxed-template-template-args, make it on by default
Needs ReviewPublic

Authored by mizvekov on Sep 9 2021, 3:37 AM.

Details

Reviewers
rsmith
Summary

A resolution to the ambiguity issues created by P0522, which is a DR solving
CWG 150, did not come as expected, so we are just going to accept the change,
and watch how users digest it.

For now we deprecate the flag with a warning, and make it on by default.
We don't remove the flag completely in order to give users a chance to
work around any problems by disabling it.

Signed-off-by: Matheus Izvekov <mizvekov@gmail.com>

Diff Detail

Event Timeline

mizvekov created this revision.Sep 9 2021, 3:37 AM
mizvekov updated this revision to Diff 371538.Sep 9 2021, 3:40 AM
mizvekov edited the summary of this revision. (Show Details)

.

mizvekov updated this revision to Diff 371539.Sep 9 2021, 3:42 AM
mizvekov edited the summary of this revision. (Show Details)

.

mizvekov published this revision for review.Sep 9 2021, 1:09 PM
mizvekov added a reviewer: rsmith.
Herald added a project: Restricted Project. · View Herald TranscriptSep 9 2021, 1:09 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
rsmith added inline comments.Wed, Sep 29, 1:12 PM
clang/www/cxx_status.html
816–817

We should list this as implemented in Clang 4, with a footnote saying that until Clang 14 you need to pass an additional flag, like we do for char8_t: https://clang.llvm.org/cxx_status.html#p0482

mizvekov updated this revision to Diff 376046.Wed, Sep 29, 2:58 PM
  • Clarify that P0522 was implemented in Clang 4, explain in footnote.
  • Add DR link.
  • C++17 section goes all green, so we fold it!
mizvekov marked an inline comment as done.Thu, Sep 30, 9:32 AM