[PATCH] D151634: [clang] Add test for CWG253

Fri Jun 2 10:43:45 PDT 2023

Endill added inline comments.

================
Comment at: clang/test/CXX/drs/dr0xx.cpp:1022

-namespace dr78 { // dr78: sup ????
+namespace dr78 { // dr78: no
   // Under DR78, this is valid, because 'k' has static storage duration, so is
----------------
shafik wrote:
> shafik wrote:
> > This is [issue 1380](https://github.com/cplusplus/papers/issues/1380) and the issue is whether we want static initialization to happen before constant initialization or whether constant initialization excludes zero-init. 
> > 
> > I think dr77 is now part of [cwg 2536](https://cplusplus.github.io/CWG/issues/2536.html) and we need to wait for the resolution of that in order to know what to do here. 
> I was mistaken and completely missed: https://eel.is/c++draft/dcl.init#general-8.sentence-2
> 
> DR 78 is just repeating what we have in: https://eel.is/c++draft/basic.start#static
> 
> The wording has changed a lot since DR 78.
Can you please elaborate how does your conclusion affect this patch? Because I feel a bit lost at this point.
Was my initial analysis correct, and we should say that this DR is not available in Clang?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151634