diff mbox series

cxx status update

Message ID a82985cb-f862-0919-4314-7d35c77333b6@acm.org
State New
Headers show
Series cxx status update | expand

Commit Message

Nathan Sidwell Jan. 4, 2021, 5:15 p.m. UTC 
Here's a patch describing the c++20 modules status.  While there I 
noticed the coroutines project status was out of date (it's done).

ok?

nathan

Comments

Gerald Pfeifer Jan. 4, 2021, 8:33 p.m. UTC | #1 
Hi Nathan,

On Mon, 4 Jan 2021, Nathan Sidwell wrote:
> Here's a patch describing the c++20 modules status.  While there I 
> noticed the coroutines project status was out of date (it's done).
> 
> ok?

you don't need extra approval for release note entries in your
areas of expertise/maintainership, but I'm always happy to help
and have a look.

I skimmed these changes and they look good.


The one part that caught my attention is

  -      <td rowspan=10> Modules </td>
  +      <td rowspan=10> Modules<br> </td>

where I'm wondering why you are adding the <br> here? 

Is this to top-align that cell since other cells in that row have 
multiple lines? If so, we should be able to generally handle this
via CSS. Check out gcc.css and the existing entry 

  table.cxxdrstatus td:nth-child(4) { text-align:center; }

there. If you want this for all rows, something like the following 
should do what you want?

  table.cxxdrstatus td:nth-child(1) { vertical-align:top; }

Or do you only want to do that for that one row? Then we could add
a new class.

Either way, if you don't want to dive in, got ahead and commit as is,
let me know what you want to accomplish, and I'll have a look. :-)

Gerald
Nathan Sidwell Jan. 4, 2021, 9:03 p.m. UTC | #2 
On 1/4/21 3:33 PM, Gerald Pfeifer wrote:
> Hi Nathan,
> 
> On Mon, 4 Jan 2021, Nathan Sidwell wrote:
>> Here's a patch describing the c++20 modules status.  While there I
>> noticed the coroutines project status was out of date (it's done).
>>
>> ok?
> 
> you don't need extra approval for release note entries in your
> areas of expertise/maintainership, but I'm always happy to help
> and have a look.
> 
> I skimmed these changes and they look good.
> 
> 
> The one part that caught my attention is
> 
>    -      <td rowspan=10> Modules </td>
>    +      <td rowspan=10> Modules<br> </td>
> 
> where I'm wondering why you are adding the <br> here?

That's an error :)  thanks for the advice

nathan
diff mbox series

Patch

diff --git i/htdocs/gcc-11/changes.html w/htdocs/gcc-11/changes.html
index f457b7e7..e044d710 100644
--- i/htdocs/gcc-11/changes.html
+++ w/htdocs/gcc-11/changes.html
@@ -228,7 +228,8 @@  a work-in-progress.</p>
   <li>The default mode has been changed to <code>-std=gnu++17</code>.</li>
   <li>Several C++20 features have been implemented:
     <ul>
-      <li>the compiler now supports <code>consteval virtual</code> functions</li>
+      <li>the compiler now supports <code>consteval virtual</code>
+	functions</li>
       <li>P2082R1, Fixing CTAD for aggregates</li>
       <li>P0593R6, Pseudo-destructors end object lifetimes</li>
       <li>P1907R1, Inconsistencies with non-type template parameters (complete
@@ -236,6 +237,9 @@  a work-in-progress.</p>
       <li>P1975R0, Fixing the wording of parenthesized aggregate-initialization</li>
       <li>P1009R2, Array size deduction in <i>new-expression</i>s</li>
       <li>P1099R5, <code>using enum</code></li>
+      <li>Modules, Requires <code>-fmodules-ts</code> and some aspects
+      are incomplete.  Refer
+      to <a href="../projects/cxx-status.html#cxx20">C++ 20 Status</a>
     </ul>
   </li>
   <li>Several C++ Defect Reports have been resolved, e.g.:
diff --git i/htdocs/projects/cxx-status.html w/htdocs/projects/cxx-status.html
index 403d6740..d8e6d8f8 100644
--- i/htdocs/projects/cxx-status.html
+++ w/htdocs/projects/cxx-status.html
@@ -418,43 +418,65 @@ 
       <td> </td>
     </tr>
     <tr>
-      <td rowspan=10> Modules </td>
+      <td rowspan=10> Modules<br> </td>
       <td><a href="https://wg21.link/p1103r3">P1103R3</a></td>
-      <td rowspan="10" class="unsupported">No (<a href="../wiki/cxx-modules">Modules Wiki</a>) </td>
-      <td rowspan="10"> </td>
+      <td class="partial"><a href="../gcc-11/changes.html#cxx">11</a> (requires <code>-fmodules-ts</code>)<br>
+	(No Private Module Fragment,<br>
+	Parser-level Global Module Entity Merging,<br>
+	Global Module Implications of <code>extern "C/C++"</code>,<br>
+	or Partition-specific Definition Visibility) </td>
+      <td> <code>__cpp_modules &gt;= 201810L</code><br> (Date of p1103r3)</td>
     </tr>
     <tr>
-      <td><a href="https://wg21.link/p1766r1">P1766R1</a></td>
+      <td><a href="https://wg21.link/p1766r1">P1766R1</a> </td>
+      <td class="unsupported center"> No </td>
+      <td> </td>
     </tr>
     <tr>
       <td><a href="https://wg21.link/p1811r0">P1811R0</a></td>
+      <td class="supported center"><a href="../gcc-11/changes.html#cxx">11</a></td>
+      <td> </td>
     </tr>
     <tr>
-      <td><a href="https://wg21.link/p1703r1">P1703R1</a></td>
+      <td><a href="https://wg21.link/p1703r1">P1703R1</a> (superceded by p1857)</td>
+      <td class="supported center"><a href="../gcc-11/changes.html#cxx">11</a></td>
+      <td> </td>
     </tr>
     <tr>
       <!-- from Belfast -->
       <td><a href="https://wg21.link/p1874r1">P1874R1</a></td>
+      <td class="supported center"><a href="../gcc-11/changes.html#cxx">11</a></td>
+      <td> </td>
     </tr>
     <tr>
       <!-- from Belfast -->
       <td><a href="https://wg21.link/p1979r0">P1979R0</a></td>
+      <td class="supported center"><a href="../gcc-11/changes.html#cxx">11</a></td>
+      <td> </td>
     </tr>
     <tr>
       <!-- from Prague -->
       <td><a href="https://wg21.link/p1779r3">P1779R3</a></td>
+      <td class="supported center"><a href="../gcc-11/changes.html#cxx">11</a></td>
+      <td> </td>
     </tr>
     <tr>
       <!-- from Prague -->
       <td><a href="https://wg21.link/p1857r3">P1857R3</a></td>
+      <td class="supported center"><a href="../gcc-11/changes.html#cxx">11</a></td>
+      <td> </td>
     </tr>
     <tr>
       <!-- from Prague -->
       <td><a href="https://wg21.link/p2115r0">P2115R0</a></td>
+      <td class="supported center"><a href="../gcc-11/changes.html#cxx">11</a></td>
+      <td> </td>
     </tr>
     <tr>
       <!-- from Prague -->
       <td><a href="https://wg21.link/p1815r2">P1815R2</a></td>
+      <td class="unsupported center"> No </td>
+      <td> </td>
     </tr>
     <tr>
       <td> Coroutines </td>
@@ -887,16 +909,16 @@ 
     <tr>
       <td> Coroutines </td>
       <td><a href="https://wg21.link/n4649">N4649</a></td>
-      <td class="unsupported"> <a href="https://gcc.gnu.org/wiki/cxx-coroutines">In progress</a> </td>
-      <td></td>
-      <td></td>
+      <td class="supported"> <a href="../gcc-10/changes.html#cxx">10</a> </td>
+      <td>-fcoroutines</td>
+      <td>__cpp_impl_coroutine &gt;= 201902L</td>
     </tr>
     <tr>
       <td> Modules </td>
       <td><a href="https://wg21.link/n4720">N4720</a></td>
-      <td class="unsupported"> <a href="https://gcc.gnu.org/wiki/cxx-modules">In progress</a> </td>
-      <td></td>
-      <td></td>
+      <td class="supported"> <a href="../gcc-11/changes.html#cxx">11</a> </td>
+      <td>-fmodules-ts</td>
+      <td>__cpp_modules &gt;= 201810L</td>
     </tr>
   </table>