shafik 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 ---------------- Endill wrote: > 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? No, so this DR was just clarifying that static objects will not have an indeterminate value if they don't have an initializer. So we can consider this NA or you can add a test with a static global w/o an init and show it has zero value. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151634/new/ https://reviews.llvm.org/D151634 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits