This is an archive of the discontinued LLVM Phabricator instance.

Differential D105024

Add papers adopted by the C++ committee in the June 2021 plenary.
ClosedPublic

Authored by cor3ntin on Jun 28 2021, 6:39 AM.

Details

Reviewers
aaron.ballman

Diff Detail

Event Timeline

cor3ntin requested review of this revision.Jun 28 2021, 6:39 AM
cor3ntin created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJun 28 2021, 6:39 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
cor3ntin added a reviewer: aaron.ballman.Jun 28 2021, 6:40 AM
cor3ntin updated this revision to Diff 354882.Jun 28 2021, 6:44 AM

Fix status of P2201

• Quuxplusone added a subscriber: • Quuxplusone.Jun 28 2021, 6:58 AM

Shouldn't the "yes/Yes" entries display Clang major version numbers instead?
Throughout, "NO" should be "No" (compare to lines 1221 and 1226).

clang/www/cxx_status.html
1287

<td><tt>if consteval</tt></td>

aaron.ballman added inline comments.Jun 28 2021, 6:58 AM
clang/www/cxx_status.html
1289

Same fix applies elsewhere

1294

You should find out which version of Clang this support was introduced in and list the version explicitly. We typically use "Yes" for "it's always been like this".

In this case, this one is Clang 13 and unreleased (rather than "full").

1319
Harbormaster completed remote builds in B111279: Diff 354882.Jun 28 2021, 7:00 AM
cor3ntin updated this revision to Diff 354891.Jun 28 2021, 7:18 AM

Fix cases

cor3ntin updated this revision to Diff 354892.Jun 28 2021, 7:20 AM

Fix status of P2156R1

cor3ntin added inline comments.Jun 28 2021, 7:34 AM
clang/www/cxx_status.html
1294

Right, I did check for the wrong thing

Harbormaster completed remote builds in B111287: Diff 354892.Jun 28 2021, 7:39 AM
aaron.ballman accepted this revision.Jun 28 2021, 7:41 AM

LGTM!

This revision is now accepted and ready to land.Jun 28 2021, 7:41 AM
aaron.ballman closed this revision.Jun 28 2021, 10:45 AM

I've commit on your behalf in 18c3c7784975700ae463bb461487d46e74324a66, thanks!

• Quuxplusone added a comment.Jun 28 2021, 11:09 AM

@cor3ntin do you have any interest in making a similar patch for the LWG issues? :)

Revision Contents

PathSize
clang/
www/
35 lines

Diff 354892

clang/www/cxx_status.html

Show First 20 Lines • Show All 1,277 Lines • ▼ Show 20 Lines <tr>
<td><a href="https://wg21.link/p1102r2">P1102R2</a></td> <td><a href="https://wg21.link/p1102r2">P1102R2</a></td>
<td class="unreleased" align="center">Clang 13</td> <td class="unreleased" align="center">Clang 13</td>
</tr> </tr>
<tr> <tr>
<td>Simpler implicit move</td> <td>Simpler implicit move</td>
<td><a href="https://wg21.link/p2266r1">P2266R1</a></td> <td><a href="https://wg21.link/p2266r1">P2266R1</a></td>
<td class="unreleased" align="center">Clang 13</td> <td class="unreleased" align="center">Clang 13</td>
</tr> </tr>
<tr>
<td><tt>if consteval</tt></td>
QuuxplusoneUnsubmitted
Not Done
ReplyInline Actions

<td><tt>if consteval</tt></td>

Quuxplusone: `<td><tt>if consteval</tt></td>`
<td><a href="https://wg21.link/P1938R3">P1938R3</a></td>
<td class="none" align="center">No</td>
aaron.ballmanUnsubmitted
Not Done
ReplyInline Actions
<td><a href="https://wg21.link/P1938R3">P1938R3</a></td>
- <td class="none" align="center">NO</td>
+ <td class="none" align="center">No</td>
</tr>

Same fix applies elsewhere

aaron.ballman: Same fix applies elsewhere
</tr>
<tr>
<td>Allow duplicate attributes</td>
<td><a href="https://wg21.link/P2156R1">P2156R1</a></td>
<td class="unreleased" align="center">Clang 13</td>
aaron.ballmanUnsubmitted
Not Done
ReplyInline Actions

You should find out which version of Clang this support was introduced in and list the version explicitly. We typically use "Yes" for "it's always been like this".

In this case, this one is Clang 13 and unreleased (rather than "full").

aaron.ballman: You should find out which version of Clang this support was introduced in and list the version…
cor3ntinAuthorUnsubmitted
Done
ReplyInline Actions

Right, I did check for the wrong thing

cor3ntin: Right, I did check for the wrong thing
</tr>
<tr>
<td>Narrowing contextual conversions to bool</td>
<td><a href="https://wg21.link/P1401R5">P1401R5</a></td>
<td class="none" align="center">No</td>
</tr>
<tr>
<td>Trimming whitespaces before line splicing</td>
<td><a href="https://wg21.link/P2223R2">P2223R2</a></td>
<td class="full" align="center">Yes</td>
</tr>
<tr>
<td>Make declaration order layout mandated</td>
<td><a href="https://wg21.link/p1847r4">P1874R4</a></td>
<td class="full" align="center">Yes</td>
</tr>
<tr>
<td>C++ identifier syntax using UAX 31</td>
<td><a href="https://wg21.link/P1949R7">P1949R7</a></td>
<td class="none" align="center">No</td>
</tr>
<tr>
<td>Mixed string literal concatenation</td>
<td><a href="https://wg21.link/p2201r1">P2201R1</a></td>
<td class="full" align="center">Yes</td>
aaron.ballmanUnsubmitted
Not Done
ReplyInline Actions
<td><a href="https://wg21.link/p2201r1">P2201R1</a></td>
- <td class="full" align="center">yes</td>
+ <td class="full" align="center">Yes</td>
</tr>
aaron.ballman:
</tr>
</table> </table>
</details> </details>
<h2 id="dr">Defect reports</h2> <h2 id="dr">Defect reports</h2>
<p>Clang generally aims to implement resolutions to Defect Reports (bug fixes <p>Clang generally aims to implement resolutions to Defect Reports (bug fixes
against prior standards) retroactively, in all prior standard versions where against prior standards) retroactively, in all prior standard versions where
the fix is meaningful. Significant Defect Report changes to language features the fix is meaningful. Significant Defect Report changes to language features
▲ Show 20 Lines • Show All 122 Lines • Show Last 20 Lines