[PATCH] D144334: [Clang] Add C++2b attribute [[assume(expression)]]

[Nikolas Klauser via Phabricator via cfe-commits]

philnik added inline comments.


================
Comment at: clang/include/clang/Basic/Attr.td:1448
+def Assume : StmtAttr {
+  let Spellings = [CXX11<"", "assume", 202302>];
+  let Documentation = [AssumeDocs];
----------------
tahonermann wrote:
> philnik wrote:
> > Izaron wrote:
> > > Honestly I don't have a clue what these numbers `202302` (apparently the `yyyymm` format) would mean :( I'd be grateful if you could help me with the right value for this.
> > https://isocpp.org/std/standing-documents/sd-6-sg10-feature-test-recommendations has generally the correct values for this. In this case it's `202207`.
> The draft standard is a better reference (http://eel.is/c++draft/cpp.cond#6). Fortunately, it agrees with SD6.
The problem with the standard is that it doesn't list the old values, so it's useless if you want to implement a paper for which the FTM has already been bumped by another paper (or you don't know whether there is another paper).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144334