dexonsmith added a comment.
Drive-by comment on the docs; otherwise this sounds awesome; as long as `else`
is easy to add later this SGTM (I'll let others do the code review).
(Although, if `else {}` and `else requires {}` would be easy to add now/soon, I
feel like it'd be worth it. Modelling an else-requires chain by hand would be
quite error-prone, and it might be annoying to stage the adoption separately...)
================
Comment at: clang/docs/Modules.rst:473
*module-declaration**
+ *requires-block**
----------------
Should this be `*requires-block-declaration*`? I'm not seeing the definition of
`*requires-block*` anywhere.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D118311/new/
https://reviews.llvm.org/D118311
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
