[PATCH] D157331: [clang] Implement C23 <stdckdint.h>

[Zijun Zhao via Phabricator via cfe-commits]

ZijunZhao added inline comments.


================
Comment at: clang/test/Headers/stdckdint.cpp:1
+// RUN: %clang_cc1 -emit-llvm -fgnuc-version=4.2.1 -std=gnu++11 %s -o - | FileCheck %s
+
----------------
enh wrote:
> ZijunZhao wrote:
> > enh wrote:
> > > hiraditya wrote:
> > > > seems like we don't have a -std=gnu23, or -std=c23 standard flag for this in clang yet.
> > > > 
> > > > https://godbolt.org/z/7dKnGEWWE
> > > > 
> > > > we probably need it before testing stdckdint i guess?
> > > other headers just use > and the previous version. (though see stdalign.h if you're looking for some random cleanup to do!)
> > > seems like we don't have a -std=gnu23, or -std=c23 standard flag for this in clang yet.
> > 
> > In the local testing, `-std=c++23` works  and all tests pass😂 
> > 
> > 
> C23 != C++23... they don't even really coordinate with one another... talk to hboehm about that some time :-)
ohhh I think `gnu++23` != `gnu23` either 😂 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157331